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

Move openidpproxyconfig to checks #2

Open
wants to merge 120 commits into
base: master
Choose a base branch
from

Conversation

p53
Copy link
Owner

@p53 p53 commented Apr 14, 2022

Title

Summary

Type

[] Bug fix
[] Feature request
[] Enhancement
[] Docs

Why?

Requirements

How to try it?

Documentation

Additional Information

Checklist:

  • My change requires a change to the documentation or CHANGELOG.
  • I have updated the documentation/CHANGELOG accordingly.

p53 and others added 30 commits December 14, 2020 22:12
* Update golangci to latest, having errors when running

* Disable some linters

* Disable some linters

Co-authored-by: Pavol Ipoth <[email protected]>
* Modify docker build files to work with new name

* Add mkdir bin to makefile docker-build

Co-authored-by: Pavol Ipoth <[email protected]>
…pelling makefile target (#14)

Co-authored-by: Pavol Ipoth <[email protected]>
* Update docu after migration to go-gatekeeper

* Remove references to louketo

* Add chat to docu

* Remove louketo from issue templates

* Add proper docker images references

* Remove docker.io from readme

* Update docu

Co-authored-by: Pavol Ipoth <[email protected]>
Fix binary name in docu
* Verify id token not access token

* Refactor to replace old oidc/jose

* Update oidc+jose library

* Remove old jose leftovers

* Fix golint errors after update oidc/jose

* Remove old oidc package

* Turn off issuer check in access token validation

* Add SkipAccessTokenClientIDCheck, SkipAccessTokenIssuerCheck flags

* Raise number of allowed func lines for golint

* Refactor getIdentity function to not return rawToken as it is already in userContext

* Remove token field from userContext, not used

* Remove token field from userContext, not used

* Remove redundant email logging

Co-authored-by: Pavol Ipoth <[email protected]>
Fix rawToken/acctoken after refreshing, it also resulted in bad x-aut…
Add new options skip-access-token-clientid-check, skip-access-token-i…
Signed-off-by: sheetalsingala <[email protected]>
Update docu to include 1.2.0 versions, fix forwarding example
Update docu to include 1.2.0 version in README
Fix splitting headers key/value when more than one equal sign there #23
* Fix SkipOpenIDProviderTLSVerify and add tests for it

* Fix linting

* Adding SkipOpenIDProviderTLSVerify for login/logout Handlers

* Add oidc tls skip verify to forwarding proxy/refreshing token

* Pass only proxyConfig to getRefreshedToken
* Fix revocation url

* Add returning http error codes when revocation fails, Add tests for revocation url

* Move revocation url variable to where it is used
Fix token encryption for loginHandler and missing refresh token cooki… #36 #37
* Refactor Dockerfile, use scratch as base image, make static binary

* Add certificates to docker image
p53 and others added 27 commits January 8, 2022 00:29
* Remove caret from go version in release github, it has different meaning
* Specify version of k3d github action to 1.5.0
* Fix TestGetRequestHostURL

* Raise token retrieval timeout in e2e test
* Raise provider timeout in e2e test
* Add E2E tests also to pull requests

* Fix Duration options, not taken by env var
* Make possible to specify minimal TLS version

* Add tests+docs for TLSMinVersion option

Co-authored-by: Jakub Onderka <[email protected]>
* fix: token expired error message

* Add tests for refresh token expiration

Co-authored-by: PraserX <[email protected]>
* Allow custom http method

* Fixed config validation, fixed config env definition

* Rewrite custom http method checks, tests

* Add docu for custom-http-methods

Co-authored-by: PraserX <[email protected]>
* Make nicer formatting in server.go

* Update package golang.org/x/text, security
* add-mock-server

* Add application mock to k8s manifest
* Add methods for validation

* Refactor config IsValid and add tests
* Refactor newOpenIDPProvider method, prepare for new features

* Update github actions
@p53 p53 self-assigned this Apr 14, 2022
@p53 p53 force-pushed the move-openidpproxyconfig-to-checks branch from ccb781e to 90a7e0d Compare April 14, 2022 18:55
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.

3 participants