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

Version Packages #94

Merged
merged 1 commit into from
Oct 10, 2023
Merged

Version Packages #94

merged 1 commit into from
Oct 10, 2023

Conversation

seek-oss-ci
Copy link
Contributor

@seek-oss-ci seek-oss-ci commented Oct 9, 2023

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.

Releases

@seek/[email protected]

Minor Changes

  • Omit request headers (#92)

    @seek/logger now omits the following properties from headers and req.headers by default:

    • x-envoy-attempt-count
    • x-envoy-decorator-operation
    • x-envoy-expected-rq-timeout-ms
    • x-envoy-external-address
    • x-envoy-internal
    • x-envoy-peer-metadata
    • x-envoy-peer-metadata-id
    • x-envoy-upstream-service-time

    To opt out of this behaviour, provide an empty list or your own list of omissible request headers to omitHeaderNames:

    const logger = createLogger({
      name: 'my-app',
    + omitHeaderNames: ['dnt', 'sec-fetch-dest'],
    });

    You can also extend the default list like so:

    - import createLogger from '@seek/logger';
    + import createLogger, { DEFAULT_OMIT_HEADER_NAMES } from '@seek/logger';
    
    const logger = createLogger({
      name: 'my-app',
    + omitHeaderNames: [...DEFAULT_OMIT_HEADER_NAMES, 'dnt', 'sec-fetch-dest']
    });

@seek-oss-ci seek-oss-ci requested a review from a team October 9, 2023 03:08
@72636c
Copy link
Member

72636c commented Oct 9, 2023

Will merge after we land #95

@72636c 72636c self-assigned this Oct 9, 2023
@seek-oss-ci seek-oss-ci force-pushed the changeset-release/master branch 2 times, most recently from dfe533d to 878802e Compare October 10, 2023 01:50
@seek-oss-ci seek-oss-ci force-pushed the changeset-release/master branch from 878802e to 89c4112 Compare October 10, 2023 01:51
@72636c 72636c enabled auto-merge (squash) October 10, 2023 01:51
@72636c 72636c merged commit 636e5f9 into master Oct 10, 2023
2 checks passed
@72636c 72636c deleted the changeset-release/master branch October 10, 2023 01:54
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