-
Notifications
You must be signed in to change notification settings - Fork 910
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lowercase scheme and authority per specification. #183
base: develop
Are you sure you want to change the base?
Conversation
@jaitaiwan I'm not sure what's going on, but it appears PT3K support never passed tests? Or isn't merged into |
We never ended up merging any py3k support into develop at all because mine was too incomplete. I don't know how it passed tests originally but it's not working now. I think the vote was on using tseaver's py3k branch and fixing the 2 tests that didn't pass. |
Somehow |
@jaitaiwan looks like you pushed PT3K builds to |
Sure not a problem. |
@joestump change has been completed and RC1 has been tagged in a release/1.9 branch :) |
Tests are passing. This is ready for a final review. 👍 |
…-oauth2 into normalize-urls-correctly
a660e24
to
ea42271
Compare
I pulled this PR into a new fork: https://github.com/TimSC/python-oauth10a |
Closes #29. More robustly follows URL normalization per the specification now. See #133 for additional and related discussion.