Skip to content

Custom request headers#636

Merged
nleush merged 15 commits intodevfrom
feature/custom-request-headers
Mar 30, 2026
Merged

Custom request headers#636
nleush merged 15 commits intodevfrom
feature/custom-request-headers

Conversation

@nleush
Copy link
Copy Markdown
Member

@nleush nleush commented Mar 17, 2026

About the changes

  • Changes to existing functionality (improvements or deprecations)

What has changed

Please describe what the code is doing (in this repo):

  • prepareRequestOptions now became async, returning promise
  • prepareRequestOptions can request additional headers from async options.getSigHeaders(uri, cb) function

Checklist

  • I have performed a self-review of my code, including Files Changed view
  • Fix/feat branches in all related repos are named the same
  • All related PRs are submitted
  • Task is DONE and links back to this PR or no task

Review

Please note any risks, caveats, migration steps, release notices or additional tasks.

  • PRs have been tested on staging by DEV
  • PRs have been tested on staging by QA
  • PRs have been tested on staging by PRODUCT

Please describe the tests and give additional instructions if required, add screenshots and links if necessary.

@nleush nleush force-pushed the feature/custom-request-headers branch from efc331f to efc0b64 Compare March 25, 2026 15:11
@nleush nleush requested a review from iparamonau March 30, 2026 13:13
@nleush nleush merged commit ec6f68e into dev Mar 30, 2026
3 checks passed
@nleush nleush deleted the feature/custom-request-headers branch March 30, 2026 13:36
nleush added a commit that referenced this pull request Apr 1, 2026
* make async `prepareRequestOptions`

* get custom request headers with `options.getSigHeaders`

* request custom headers when `options.sig`

* catch error in `getSigHeaders`

* log getSigHeaders error

* update user agent workaround

* tests: allow use getSigHeaders

* use getSigHeaders

* debugger: use `getSigHeaders`

* bugfix import

* bugfix `options undefined`

* fix user agent assign priority

* Update audit.log

* remove debug log

* remove debug log
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.

2 participants