SUSE / Portus

Authorization service and frontend for Docker registry (v2)
http://port.us.org/
Apache License 2.0
3k stars 471 forks source link

auth: remove unused address scope from openid connect #2233

Closed bonifaido closed 4 years ago

bonifaido commented 4 years ago

Signed-off-by: Nandor Kracser bonifaido@gmail.com

Summary

I just set up Portus with OpenID Connect authentication using Dex which doesn't support the address scope. I just checked through the code and data returned from that scope is not used at all, so I have removed it from the OAuth request.

bonifaido commented 4 years ago

@mssola could you please check this?

stale[bot] commented 4 years ago

Thanks for all your contributions! This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

SuperSandro2000 commented 4 years ago

Spam bot no. bad boy.

stale[bot] commented 4 years ago

Thanks for all your contributions! This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

SuperSandro2000 commented 4 years ago

bump

stale[bot] commented 4 years ago

Thanks for all your contributions! This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.