Skip to content
This repository was archived by the owner on Jan 28, 2025. It is now read-only.

feat(nextjs-component): forward common headers such as authorization … #1660

Merged
merged 1 commit into from
Sep 9, 2021

Conversation

dphang
Copy link
Collaborator

@dphang dphang commented Sep 9, 2021

…and host by default

  • Also update rewrite e2e tests

@slsnextbot
Copy link
Collaborator

Handler Size Report

No changes to handler sizes.

Base Handler Sizes (kB) (commit 67c280f)

{
    "Lambda@Edge": {
        "Default Lambda": {
            "Standard": 1253,
            "Minified": 448
        },
        "API Lambda": {
            "Standard": 100,
            "Minified": 44
        },
        "Image Lambda": {
            "Standard": 918,
            "Minified": 368
        },
        "Regeneration Lambda": {
            "Standard": 645,
            "Minified": 235
        }
    }
}

New Handler Sizes (kB) (commit 65268fd)

{
    "Lambda@Edge": {
        "Default Lambda": {
            "Standard": 1253,
            "Minified": 448
        },
        "API Lambda": {
            "Standard": 100,
            "Minified": 44
        },
        "Image Lambda": {
            "Standard": 918,
            "Minified": 368
        },
        "Regeneration Lambda": {
            "Standard": 645,
            "Minified": 235
        }
    }
}

@codecov
Copy link

codecov bot commented Sep 9, 2021

Codecov Report

Merging #1660 (65268fd) into master (67c280f) will decrease coverage by 0.09%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1660      +/-   ##
==========================================
- Coverage   84.23%   84.14%   -0.10%     
==========================================
  Files          98       98              
  Lines        3444     3449       +5     
  Branches     1022     1027       +5     
==========================================
+ Hits         2901     2902       +1     
- Misses        483      485       +2     
- Partials       60       62       +2     
Impacted Files Coverage Δ
...rless-components/nextjs-component/src/component.ts 87.26% <33.33%> (-1.29%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 67c280f...65268fd. Read the comment docs.

@dphang dphang merged commit 793f0a2 into master Sep 9, 2021
@delete-merged-branch delete-merged-branch bot deleted the dphang/rewrite-test-coverage branch September 9, 2021 20:47
dphang added a commit that referenced this pull request Sep 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants