Skip to content
This repository has been archived by the owner on Aug 9, 2022. It is now read-only.

[Not ready to merge] forward extra needed headers to headless browser #369

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

zhongnansu
Copy link
Member

@zhongnansu zhongnansu commented May 18, 2021

Issue #, if available:

Description of changes:

  • Instead of only forwarding the security_authentication, this change will forward all cookies to handle different security setup
  • rewrite the logic of 0160a02 to pick up proxy-auth headers
  • add extra required headers to handle different security setting.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@zhongnansu zhongnansu added the need migrate need migrate to OpenSearch label May 18, 2021
@zhongnansu zhongnansu marked this pull request as ready for review May 18, 2021 01:15
@zhongnansu zhongnansu changed the title forward extra needed headers to headless browser [Not ready to merge]forward extra needed headers to headless browser May 19, 2021
@zhongnansu zhongnansu changed the title [Not ready to merge]forward extra needed headers to headless browser **[Not ready to merge]** forward extra needed headers to headless browser May 19, 2021
@zhongnansu zhongnansu changed the title **[Not ready to merge]** forward extra needed headers to headless browser [Not ready to merge] forward extra needed headers to headless browser May 19, 2021
@zhongnansu zhongnansu marked this pull request as draft June 9, 2021 20:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
need migrate need migrate to OpenSearch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants