Skip to content
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

Documentation: Azure oauth2 scope must include "openid" #304

Closed
Pavlos1 opened this issue Aug 15, 2024 · 1 comment · Fixed by #306
Closed

Documentation: Azure oauth2 scope must include "openid" #304

Pavlos1 opened this issue Aug 15, 2024 · 1 comment · Fixed by #306
Labels
✨ enhancement New feature or request

Comments

@Pavlos1
Copy link

Pavlos1 commented Aug 15, 2024

Problem Statement

First: I like this project a lot, I was considering getting my company to purchase Cisco AnyConnect before I found this and it's really nice to have an open source alternative. I finally managed to get everything set up and I have a suggestion to improve the documentation:

In order for openvpn-auth-oauth2 to work with Azure, it needs to receive an id_token. This doesn't happen unless CONFIG_OAUTH2_SCOPES=openid is set in /etc/sysconfig/openvpn-auth-oauth2. It would help if this information was provided in the wiki. (Maybe it is and I just didn't find it?)

Proposed Solution

No response

Additional information

No response

Acceptance Criteria

No response

@Pavlos1 Pavlos1 added the ✨ enhancement New feature or request label Aug 15, 2024
@jkroepke
Copy link
Owner

jkroepke commented Aug 18, 2024

Hey @Pavlos1

I really appreciate your donation! I directly convert it to some beer with friends. I'm happy to hear, that this open source project avoids at least one more AnyConnect installation. Let me know, if you need some help.

You are right. The scope are currently not documented. If unconfigured by used, openvpn-auth-oauth2 choices the correct defaults. #306 will enrich the documents here to cover your case.

https://github.com/jkroepke/openvpn-auth-oauth2/pull/306/files#diff-e91aedfc8ff498d1b7a6c00656928af0a769da0cba067e9c0ef249105b688c4eR40-R42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants