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

fix: Relay Auth header via Query Params on Redirect #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nsrCodes
Copy link
Contributor

@nsrCodes nsrCodes commented Apr 4, 2024

fixes: requestly/requestly-desktop-app#71

Also adds the following improvements:

  • makes sure that the logs are now built from ctx.rq.original_request and not ctx.clientToProxyRequest
  • improves the getQueryParamsMap by using the qs module now

@nsrCodes nsrCodes requested a review from wrongsahil April 4, 2024 02:23
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.

Authorization header is not passed with Redirect Request or Replace String
1 participant