Skip to content

OAuth1: Allow IPv6 addresses being parsed by signature - #818

Merged
JonathanHuot merged 5 commits into
oauthlib:masterfrom
dasm:master
Sep 6, 2022
Merged

JonathanHuot merged 5 commits into
oauthlib:masterfrom
dasm:master

Conversation

@dasm

@dasm dasm commented May 19, 2022

Copy link
Copy Markdown
Contributor

This PR addresses issue with incorrectly parsing IPv6 address,
described here: #817

This PR addresses issue with incorrectly parsing IPv6 address,
described here: oauthlib#817
@dasm

dasm commented May 19, 2022

Copy link
Copy Markdown
Contributor Author

It seems like it won't be so easy fix.
If I understand RFC correctly:https://www.rfc-editor.org/rfc/rfc3986#section-3.2 authority should be allowed to have IPs as well.

Does it mean base_string_uri should be checking only for domain names and not IPs?

Comment thread oauthlib/oauth1/rfc5849/signature.py Outdated
Comment thread tests/oauth1/rfc5849/test_signatures.py
@dasm

dasm commented Jun 15, 2022 •

Copy link
Copy Markdown
Contributor Author

Bandit failure seems to be unrelated with my changes.

@auvipy

auvipy commented Jun 22, 2022

Copy link
Copy Markdown
Member

it is apparent that the CI is not working, right?

@dasm

dasm commented Jun 22, 2022

Copy link
Copy Markdown
Contributor Author

@auvipy how can I rekick CI?

@xek xek left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks!

@auvipy
auvipy self-requested a review June 28, 2022 12:00
@dasm

dasm commented Jul 6, 2022

Copy link
Copy Markdown
Contributor Author

Hey @auvipy ! Do you have any feedback on this?
Thanks!

@auvipy auvipy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like to introduce new CI

@dasm

dasm commented Jul 6, 2022

Copy link
Copy Markdown
Contributor Author

Sure. Do you need any help?

@dasm

dasm commented Jul 20, 2022

Copy link
Copy Markdown
Contributor Author

Hi @auvipy
Can I help somehow? How is it going?

@auvipy

auvipy commented Jul 20, 2022

Copy link
Copy Markdown
Member

we need to setup a new CI

@dasm

dasm commented Aug 25, 2022

Copy link
Copy Markdown
Contributor Author

Hi @auvipy!
How is it going with a new CI? Do you need any help with setting that up?

@d34dh0r53 d34dh0r53 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@auvipy auvipy added this to the 3.2.1 milestone Aug 31, 2022
@JonathanHuot
JonathanHuot merged commit b4bdd09 into oauthlib:master Sep 6, 2022
@dasm

dasm commented Sep 6, 2022

Copy link
Copy Markdown
Contributor Author

Thanks @JonathanHuot !

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants