Skip to content
This repository was archived by the owner on Jun 25, 2026. It is now read-only.

Add user agent and bump dependencies so that cargo install works again - #268

Closed
pine-free wants to merge 6 commits into
Builditluc:mainfrom
pine-free:feature/user-agent
Closed

pine-free wants to merge 6 commits into
Builditluc:mainfrom
pine-free:feature/user-agent

Conversation

@pine-free

Copy link
Copy Markdown

The package was broken for me, the requests to wikipedia's api were returning with 403 forbidden because they had no user agent set, and cargo install wiki-tui did not work because there were 2 versions of ratatui present

I bumped the versions and the rust version, and added a user agent, as well as a general "wiki client" scaffolding for use throughout the package

@pine-free

Copy link
Copy Markdown
Author

This resolves #265 and #267

@AetherSky-arch

AetherSky-arch commented Sep 11, 2025 •

Copy link
Copy Markdown

Tested on Ubuntu 22.04.
This does not seem to fix #267, still works for #265 tho

@pine-free

Copy link
Copy Markdown
Author

Tested on Ubuntu 22.04. This does not seem to fix #267, still works for #265 tho

Huh, I tested on Ubuntu 24 and it did work. Will look into it. Thanks for the report

@AetherSky-arch

Copy link
Copy Markdown

Was worried I had reported an error caused by an outdated system, so I tested it on an up-to-date Arch machine.

Can confirm #267 remains unresolved.

@EvilsSouls

Copy link
Copy Markdown

Was worried I had reported an error caused by an outdated system, so I tested it on an up-to-date Arch machine.

Can confirm #267 remains unresolved.

It worked for me when I switched to the correct branch (feature/user-agent) before running cargo build --release.

@Builditluc

Copy link
Copy Markdown
Owner

The issue has been fixed by #273

@Builditluc Builditluc closed this Dec 2, 2025
@github-project-automation github-project-automation Bot moved this from Todo to Done in wiki-tui: Roadmap Dec 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants