-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #2213 from kvanzuijlen/release-v7.5.0
Release v7.5.0
- Loading branch information
Showing
17 changed files
with
2,317 additions
and
16 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| --- | ||
| id: behaviour | ||
| title: Behaviour | ||
| --- | ||
|
|
||
| 1. Any request passing through the proxy (and not matched by `--skip-auth-regex`) is checked for the proxy's session cookie (`--cookie-name`) (or, if allowed, a JWT token - see `--skip-jwt-bearer-tokens`). | ||
| 2. If authentication is required but missing then the user is asked to log in and redirected to the authentication provider (unless it is an Ajax request, i.e. one with `Accept: application/json`, in which case 401 Unauthorized is returned) | ||
| 3. After returning from the authentication provider, the oauth tokens are stored in the configured session store (cookie, redis, ...) and a cookie is set | ||
| 4. The request is forwarded to the upstream server with added user info and authentication headers (depending on the configuration) | ||
|
|
||
| Notice that the proxy also provides a number of useful [endpoints](features/endpoints.md). |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,49 @@ | ||
| --- | ||
| id: security | ||
| title: Security | ||
| --- | ||
|
|
||
| :::note | ||
| OAuth2 Proxy is a community project. | ||
| Maintainers do not work on this project full time, and as such, | ||
| while we endeavour to respond to disclosures as quickly as possible, | ||
| this may take longer than in projects with corporate sponsorship. | ||
| ::: | ||
|
|
||
| ## Security Disclosures | ||
|
|
||
| :::important | ||
| If you believe you have found a vulnerability within OAuth2 Proxy or any of its | ||
| dependencies, please do NOT open an issue or PR on GitHub, please do NOT post | ||
| any details publicly. | ||
| ::: | ||
|
|
||
| Security disclosures MUST be done in private. | ||
| If you have found an issue that you would like to bring to the attention of the | ||
| maintenance team for OAuth2 Proxy, please compose an email and send it to the | ||
| list of maintainers in our [MAINTAINERS](https://github.com/oauth2-proxy/oauth2-proxy/blob/master/MAINTAINERS) file. | ||
|
|
||
| Please include as much detail as possible. | ||
| Ideally, your disclosure should include: | ||
| - A reproducible case that can be used to demonstrate the exploit | ||
| - How you discovered this vulnerability | ||
| - A potential fix for the issue (if you have thought of one) | ||
| - Versions affected (if not present in master) | ||
| - Your GitHub ID | ||
|
|
||
| ### How will we respond to disclosures? | ||
|
|
||
| We use [GitHub Security Advisories](https://docs.github.com/en/github/managing-security-vulnerabilities/about-github-security-advisories) | ||
| to privately discuss fixes for disclosed vulnerabilities. | ||
| If you include a GitHub ID with your disclosure we will add you as a collaborator | ||
| for the advisory so that you can join the discussion and validate any fixes | ||
| we may propose. | ||
|
|
||
| For minor issues and previously disclosed vulnerabilities (typically for | ||
| dependencies), we may use regular PRs for fixes and forego the security advisory. | ||
|
|
||
| Once a fix has been agreed upon, we will merge the fix and create a new release. | ||
| If we have multiple security issues in flight simultaneously, we may delay | ||
| merging fixes until all patches are ready. | ||
| We may also backport the fix to previous releases, | ||
| but this will be at the discretion of the maintainers. |
Oops, something went wrong.