Skip to content

[7.x] introduce pre-/post-auth request hooks for HttpServer (#36690)#38265

Merged
mshustov merged 2 commits intoelastic:7.xfrom
mshustov:backport/7.x/pr-36690
Jun 7, 2019
Merged

[7.x] introduce pre-/post-auth request hooks for HttpServer (#36690)#38265
mshustov merged 2 commits intoelastic:7.xfrom
mshustov:backport/7.x/pr-36690

Conversation

@mshustov
Copy link
Contributor

@mshustov mshustov commented Jun 6, 2019

Backports the following commits to 7.x:

* introduce pre-,post-auth stages

* cleanup integration_tests. now contracts available in tests

* auth per route is configurable

* Unify lifecycle results structure

* expose api to store auth state and status via http service

* update tests

* update docs

* use full name, auth should not mutate request

* move basePath functionality under namespace

* regenerate docs

* Revert "move basePath functionality under namespace"

This reverts commit 9599d32.

* Revert "regenerate docs"

This reverts commit 1799d3b.

* regenerate docs

* updated yarn.lock no idea why

* extract AuthStateStorage to a separate entity

* get rid of nested ifs

* describe what is the difference between hooks

* re-wording
@mshustov mshustov added the backport This PR is a backport of another PR label Jun 6, 2019
@elasticmachine
Copy link
Contributor

💔 Build Failed

@mshustov
Copy link
Contributor Author

mshustov commented Jun 6, 2019

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@mshustov mshustov merged commit 3278cd7 into elastic:7.x Jun 7, 2019
@mshustov mshustov deleted the backport/7.x/pr-36690 branch June 7, 2019 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants