requests_auth icon indicating copy to clipboard operation
requests_auth copied to clipboard

Allow to provide FQDN for localhost redirect

Open ManojSN opened this issue 2 years ago • 3 comments

BrowserAuth currently is hardcoded to use http://localhost. Prefer to pass the request_uri_domain as a keyword arg, with default being http://localhost.

ManojSN avatar Oct 26 '22 02:10 ManojSN

Do you mean that you have some kind of alias for localhost? The token needs to be received in some way, how would this work?

Colin-b avatar Oct 26 '22 16:10 Colin-b

Hi Colin,

Thanks for responding, I was going to send you a PR.

The request is to allow the implementor to be able to provide a fqdn for local host.

http://localhost.somecompany.com

I just sent a PR for your initial comments

Manoj Nair

Thanks

Manoj

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: Colin Bounouar @.> Sent: Wednesday, October 26, 2022 12:25:28 PM To: Colin-b/requests_auth @.> Cc: ManojSN @.>; Author @.> Subject: Re: [Colin-b/requests_auth] Allow domain name to be provided to authentication.BrowserAuth (Issue #74)

Do you mean that you have some kind of alias for localhost? The token needs to be received in some way, how would this work?

— Reply to this email directly, view it on GitHubhttps://github.com/Colin-b/requests_auth/issues/74#issuecomment-1292299949, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACKSJAZD36LYD6HGWPBLWQTWFFLPRANCNFSM6AAAAAAROQ6MEU. You are receiving this because you authored the thread.Message ID: @.***>

ManojSN avatar Oct 26 '22 17:10 ManojSN

Hi Colin,

As for the SonarCode analysis, guessing only you can accept the risk with hardcoded http://

Manoj

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: Manoj Nair @.> Sent: Wednesday, October 26, 2022 1:33:09 PM To: Colin-b/requests_auth @.> Subject: Re: [Colin-b/requests_auth] Allow domain name to be provided to authentication.BrowserAuth (Issue #74)

Hi Colin,

Thanks for responding, I was going to send you a PR.

The request is to allow the implementor to be able to provide a fqdn for local host.

http://localhost.somecompany.com

I just sent a PR for your initial comments

Manoj Nair

Thanks

Manoj

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: Colin Bounouar @.> Sent: Wednesday, October 26, 2022 12:25:28 PM To: Colin-b/requests_auth @.> Cc: ManojSN @.>; Author @.> Subject: Re: [Colin-b/requests_auth] Allow domain name to be provided to authentication.BrowserAuth (Issue #74)

Do you mean that you have some kind of alias for localhost? The token needs to be received in some way, how would this work?

— Reply to this email directly, view it on GitHubhttps://github.com/Colin-b/requests_auth/issues/74#issuecomment-1292299949, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACKSJAZD36LYD6HGWPBLWQTWFFLPRANCNFSM6AAAAAAROQ6MEU. You are receiving this because you authored the thread.Message ID: @.***>

ManojSN avatar Oct 26 '22 21:10 ManojSN

Version 8.0.0 is available on pypi and fixes this issue

Colin-b avatar Jun 18 '24 18:06 Colin-b