search

swift-server / swift-aws-lambda-runtime

Swift implementation of AWS Lambda Runtime
Apache License 2.0
1.13k stars 102 forks source link

[RFC] Adopts *readonly* BaggageContext (Meh for composition) #168

Closed ktoso closed 3 years ago

ktoso commented 3 years ago

This is a "minimal changes" adoption of Context.

It is sub optimal I believe, and we should rather adopt: https://github.com/swift-server/swift-aws-lambda-runtime/pull/167

I post both so they can be compared. The CoW style implementation in #167 is much better and I believe we should adopt that, AND the Context library should adopt get/set on the Context.baggage. See that issue for more details.

ktoso commented 3 years ago

We decided readonly sucks and https://github.com/swift-server/swift-aws-lambda-runtime/pull/167 rocks.