Skip to content

Conversation

@lfarrel6
Copy link
Member

@lfarrel6 lfarrel6 commented Feb 17, 2025

Why

We want to extend support for request filtering when sending via forward proxy to account for the path if specific on the decryptionDomains.

How

  • Add domainTarget which builds a small 'target' object from the provided list of domains. The target object wraps the ability to match a request based on hostname and optionally target
  • Update all request intercept logic to include the domain and path
  • Update the jsdoc typing comments to be more accurate.
  • Tidy up some unused code

@changeset-bot
Copy link

changeset-bot bot commented Feb 17, 2025

🦋 Changeset detected

Latest commit: 880d365

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@evervault/sdk Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

boilsquid
boilsquid previously approved these changes Feb 17, 2025
@lfarrel6 lfarrel6 marked this pull request as ready for review February 17, 2025 16:48
@lfarrel6 lfarrel6 merged commit 720476d into master Feb 17, 2025
10 checks passed
@lfarrel6 lfarrel6 deleted the feat/support-path-based-filtering-of-domains branch February 17, 2025 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants