Skip to content

Escape urls for the AUR#424

Merged
fosskers merged 1 commit intofosskers:masterfrom
theotherjimmy:escape-urls
Oct 10, 2016
Merged

Escape urls for the AUR#424
fosskers merged 1 commit intofosskers:masterfrom
theotherjimmy:escape-urls

Conversation

@theotherjimmy
Copy link
Contributor

should resolve #421

@theotherjimmy
Copy link
Contributor Author

@bb010g, could you review?

@theotherjimmy
Copy link
Contributor Author

@willfrew, could you test?

@fosskers
Copy link
Owner

Wreq wasn't handling that automatically?

@theotherjimmy
Copy link
Contributor Author

nope. Strange.

@theotherjimmy
Copy link
Contributor Author

@fosskers, CI done. Want to merge?

@fosskers
Copy link
Owner

Ah, I see. We're only grabbing PKGBUILDs the old way via wreq, and all other RPC calls are via servant in the aur library.

@fosskers fosskers merged commit 4a5a91e into fosskers:master Oct 10, 2016
@fosskers
Copy link
Owner

Thanks for this, guys.

@willfrew
Copy link

Legend(s) - cheers :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bad handling of packages with the '+' character in their name

3 participants