Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

feat: add subscriptionsMatcher Lambda #559

Merged
merged 11 commits into from
Feb 17, 2022

Conversation

carvantes
Copy link
Contributor

@carvantes carvantes commented Feb 10, 2022

Description of changes:
Create Lambda function and connect it to the DDB stream

  • Add subscriptionsMatcherLambda
  • Update folder structure to put Matcher, Reaper and RestHook Lambdas in separate folders under src/subscriptions
  • Only enable Matcher and Reaper Lambda when subscriptionsEnabled is true
  • Put SDK3 SNS Client as externals in bundle as it's not compatible with webpack

Checklist:

  • Have you successfully deployed to an AWS account with your changes?
  • Have you written new tests for your core changes, as applicable?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@carvantes carvantes requested a review from a team as a code owner February 10, 2022 08:39
@codecov-commenter
Copy link

codecov-commenter commented Feb 10, 2022

Codecov Report

Merging #559 (085dd76) into rest-hook (69128d7) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           rest-hook     #559   +/-   ##
==========================================
  Coverage      94.84%   94.84%           
==========================================
  Files              1        1           
  Lines             97       97           
  Branches          14       14           
==========================================
  Hits              92       92           
  Misses             5        5           

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 69128d7...085dd76. Read the comment docs.

Base automatically changed from rest-hook to feat-subscriptions February 11, 2022 15:19
@github-actions github-actions bot added size/xl and removed size/m labels Feb 14, 2022
@Bingjiling Bingjiling changed the title wip: add subscriptionsMatcher Lambda feat: add subscriptionsMatcher Lambda Feb 15, 2022
@github-actions github-actions bot added size/m and removed size/xl labels Feb 17, 2022
@carvantes carvantes merged commit 1ef9172 into feat-subscriptions Feb 17, 2022
@carvantes carvantes deleted the dev-wip-su-bmatcher-lambda branch February 17, 2022 20:37
carvantes added a commit that referenced this pull request Mar 7, 2022
* feat: add GSI to Resource DDB Table (#533)

* feat: Add data validation for subscription (#543)

* fix: remove _subsciptionStatus from export result field (#555)

* feat: sns, sqs, dlq for Subscriptions  (#554)

* feat: Rest hook Lambda (#558)

* feat: subscriptionReaper (#557)

* feat: add subscriptionsMatcher Lambda (#559)

* test: Add Subscriptions test infrastructure/helper (#569)

* fix: update unit tests for subscription reaper (#567)

* test: add subscriptions env vars in gh actions (#572)

* fix: encrypt logs for new Lambda fns (#574)

* test: add Subscription reaper tests (#575)

* feat: emit end to end latency metric from rest-hook Lambda (#570)

* test: add tests for tenant isolation of subscriptions (#577)

* feat: add DLQ for matcher Lambda (#576)

* test: add end to end tests for subscriptions (#578)

* perf: partial failures for restHook Lambda (#579)

* docs: add Subscription docs (#582)

Co-authored-by: Sukeerth Vegaraju <ssvegaraju@yahoo.co.in>
Co-authored-by: zheyanyu <zheyanyu@amazon.com>
Co-authored-by: Yanyu Zheng <yz2690@columbia.edu>
Co-authored-by: brndhpkn <98061326+brndhpkn@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants