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

Update Fork #1

Merged
merged 22 commits into from
Sep 30, 2020
Merged

Update Fork #1

merged 22 commits into from
Sep 30, 2020

Conversation

Bibob7
Copy link
Owner

@Bibob7 Bibob7 commented Sep 30, 2020

Description

Motivation and Context

How Has This Been Tested?

Checklist:

  • My change requires a change to the documentation or CHANGELOG.
  • I have updated the documentation/CHANGELOG accordingly.
  • I have created a feature (non-master) branch for my PR.

stefansedich and others added 22 commits September 21, 2020 10:43
Add support to ensure user belongs in required groups when using the OIDC provider
docs: fix Keycloak provider documentation
Stop accepting legacy SHA1 signed cookies
Redis configuration startup validation
It isn't used in any providers and we have future plans
to remove the specialness of PreferredUsername and make it
an optional field in the session.

User, Email & Groups will eventually be the only first class
fields on the session that are always set.
…e-getter

Remove provider GetPreferredUsername getter method
…ltGetLoginURL

This allows us to reuse code from different providers in case slight
modifications to the URL are needed.
Also add unit test to ensure logingov specific logic is applied.
And don't require the caller to know how to use the returned params.
Pass resource parameter in login url
* fix import path for v7

find ./ -name "*.go" | xargs sed -i -e 's|"github.com/oauth2-proxy/oauth2-proxy|"github.com/oauth2-proxy/oauth2-proxy/v7|'

* fix module path

* go mod tidy

* fix installation docs

* update CHANGELOG

* Update CHANGELOG.md

Co-authored-by: Joel Speed <[email protected]>

Co-authored-by: Joel Speed <[email protected]>
@Bibob7 Bibob7 merged commit d4745fd into Bibob7:master Sep 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants