Skip to content

chore: remove v2/aws-sam and update the Lambda Web Adapter - #5303

Merged
ReneWerner87 merged 1 commit into
masterfrom
claude/remove-archived-lambda-proxy
Aug 1, 2026
Merged

chore: remove v2/aws-sam and update the Lambda Web Adapter#5303
ReneWerner87 merged 1 commit into
masterfrom
claude/remove-archived-lambda-proxy

Conversation

@ReneWerner87

@ReneWerner87 ReneWerner87 commented Aug 1, 2026

Copy link
Copy Markdown
Member

Background

github.com/awslabs/aws-lambda-go-api-proxy was archived on 21 May 2025. The maintainers deprecated it in favour of Lambda Web Adapter:

I would like to deprecate this library in favor of Lambda Web Adapter

Lambda Web Adapter is a Lambda layer or container extension, not a Go library, so there is no drop-in replacement import. And since the archived library's Fiber adapter only ever supported Fiber v2, there will never be a v3 version of it.

That adapter was the only reason v2/aws-sam was still on Fiber v2, and that recipe was the only thing keeping the v2/ directory alive.

Why remove instead of port

aws-sam-container already covers the same ground on current dependencies:

v2/aws-sam aws-sam-container
Fiber v2.52.14 v3.4.0
Adapter archived library, /fiber Lambda Web Adapter
Packaging zip container image
Adapter code in the recipe yes none, just app.Listen

Porting v2/aws-sam to Lambda Web Adapter would have produced a near-duplicate of aws-sam-container whose only distinguishing feature is zip instead of container packaging. It was also never listed in the README recipe index.

Removing it takes the archived dependency and the last Fiber v2 recipe out of the repository, and empties v2/.

Dropped along with it: a dead replace gopkg.in/yaml.v2 => v2.2.8 in that recipe's go.mod. go mod why reported main module does not need module gopkg.in/yaml.v2.

README index

No change needed, verified rather than assumed. v2/aws-sam was never linked from the root README.md, which has no reference to v2/ at all and no nested-path links. After this removal the index is exactly consistent: 87 links, 87 recipe directories, no dead links and no unlisted recipes.

Lambda Web Adapter bump

aws-sam-container pinned aws-lambda-adapter:0.7.0. Updated to 1.1.0.

Verified against the ECR Public registry rather than assumed:

  • 1.1.0 is the newest tag (1.1.0, 1.1.0-aarch64, 1.1.0-x86_64, ahead of 1.0.1)
  • it is a manifest.list.v2 covering linux/amd64 and linux/arm64, matching how the Dockerfile consumes it
  • upstream still documents the same /lambda-adapter source path for the 1.x line, so the COPY --from is unchanged apart from the tag

The image build itself was not exercised, since no Docker daemon is available in this environment. Worth a local docker build before merging.

Remaining Fiber v2 references

Both are unaffected and need no action. Each resolves to v2.52.14, the latest release with no open advisories:

  • openapi pulls it in through huma/v2/adapters/humafiber, which targets Fiber v2 upstream
  • svelte-netlify has it only as an unreachable module-graph entry, with no go.sum entry and go mod why reporting main module does not need module

svelte-netlify still depends on the archived library for its core package. That is handled in #5304.

Summary by CodeRabbit

  • Updates

    • Updated the AWS Lambda Web Adapter image to version 1.1.0 for container builds.
  • Removed

    • Removed the AWS SAM example application and its documentation, including setup, local invocation, deployment, and configuration guidance.

github.com/awslabs/aws-lambda-go-api-proxy was archived on 21 May 2025.
The maintainers deprecated it in favour of Lambda Web Adapter, which is a
Lambda layer or container extension rather than a Go library, so there is
no drop-in replacement import and no path to a Fiber v3 adapter. The
library's Fiber adapter only ever supported Fiber v2, which is the sole
reason this recipe was still on Fiber v2 and the only thing keeping the
v2/ directory alive.

aws-sam-container already covers the same ground on current dependencies:
SAM, Fiber v3, Lambda Web Adapter, and no adapter code at all, just
app.Listen. Porting v2/aws-sam would have produced a near-duplicate whose
only distinguishing feature is zip instead of container packaging, so it
is removed rather than migrated. It was not listed in the README recipe
index either.

Removing it takes the archived dependency and the last Fiber v2 recipe
out of the repository, and empties the v2/ directory.

Also bumps the Lambda Web Adapter in aws-sam-container from 0.7.0 to
1.1.0. Verified against the ECR Public registry: 1.1.0 is the newest tag
and is a multi-arch manifest list covering linux/amd64 and linux/arm64,
and upstream still documents the same /lambda-adapter source path for the
1.x line. The image build itself was not exercised here, since no Docker
daemon is available in this environment.

The two remaining Fiber v2 references are unaffected and need no action:
openapi pulls it in through huma's humafiber adapter, and svelte-netlify
only has it as an unreachable module-graph entry. Both resolve to
v2.52.14, the latest release with no open advisories.
Copilot AI review requested due to automatic review settings August 1, 2026 11:42
@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The AWS SAM container image now uses AWS Lambda Web Adapter version 1.1.0. The standalone AWS SAM example README and application entry point were deleted.

Changes

AWS SAM recipe updates

Layer / File(s) Summary
Lambda adapter image update
aws-sam-container/app/Dockerfile
The Dockerfile now copies AWS Lambda Web Adapter version 1.1.0 instead of 0.7.0.
Remove standalone SAM example
v2/aws-sam/README.md, v2/aws-sam/app/main.go
The AWS SAM README and Fiber Lambda application were deleted.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: copilot

Poem

I’m a rabbit with a freshly packed crate,
The adapter hops to version one-point-eight—
One-point-one, precise and bright,
Old SAM pages vanish from sight.
Hop, build, deploy, goodnight!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes both main changes: removing v2/aws-sam and updating the Lambda Web Adapter.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/remove-archived-lambda-proxy

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the legacy v2/aws-sam recipe that depended on the archived aws-lambda-go-api-proxy Fiber v2 adapter, and updates the aws-sam-container recipe to use a newer Lambda Web Adapter image tag.

Changes:

  • Deleted the entire v2/aws-sam SAM zip-based example (template, config, docs, and Go module).
  • Updated aws-sam-container to copy the Lambda Web Adapter extension from public.ecr.aws/awsguru/aws-lambda-adapter:1.1.0.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
v2/aws-sam/template.yaml Removed SAM template for the deprecated v2 recipe.
v2/aws-sam/samconfig.toml Removed SAM CLI config for the deprecated v2 recipe.
v2/aws-sam/README.md Removed documentation for the deprecated v2 recipe.
v2/aws-sam/app/main.go Removed Lambda handler using the archived aws-lambda-go-api-proxy Fiber adapter.
v2/aws-sam/app/go.mod Removed Go module for the deprecated v2 recipe (including archived dependency).
v2/aws-sam/app/go.sum Removed Go dependency lockfile for the deprecated v2 recipe.
aws-sam-container/app/Dockerfile Bumped the Lambda Web Adapter image tag used for the extension layer.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ReneWerner87
ReneWerner87 enabled auto-merge August 1, 2026 11:49
@ReneWerner87
ReneWerner87 disabled auto-merge August 1, 2026 11:50
@ReneWerner87
ReneWerner87 enabled auto-merge August 1, 2026 11:51
@ReneWerner87
ReneWerner87 disabled auto-merge August 1, 2026 11:59
@ReneWerner87
ReneWerner87 merged commit 2d190a1 into master Aug 1, 2026
10 checks passed
@ReneWerner87
ReneWerner87 deleted the claude/remove-archived-lambda-proxy branch August 1, 2026 11:59
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.

3 participants