Feat/snyk fixes test #2633

Sanjay-Veernala · 2023-09-20T06:37:54Z

Description of the change

Merged all the open snyk upgarde PRs into one.

Description here

Type of change

Bug fix (non-breaking change that fixes an issue)
New feature (non-breaking change that adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

Fix #1

Checklists

Development

Lint rules pass locally
The code changed/added as part of this pull request has been covered with tests
All tests related to the changed code pass in development

Code review

This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
"Ready for review" label attached to the PR and reviewers mentioned in a comment
Changes have been reviewed by at least one other engineer
Issue from task tracker has a link to this pull request

Snyk has created this PR to upgrade @aws-sdk/lib-storage from 3.370.0 to 3.391.0. See this package in npm: https://www.npmjs.com/package/@aws-sdk/lib-storage See this project in Snyk: https://app.snyk.io/org/datamanagement/project/39f686fe-9e69-4343-a9ce-129e22bff288?utm_source=github&utm_medium=referral&page=upgrade-pr

Snyk has created this PR to upgrade @aws-sdk/credential-providers from 3.363.0 to 3.391.0. See this package in npm: https://www.npmjs.com/package/@aws-sdk/credential-providers See this project in Snyk: https://app.snyk.io/org/datamanagement/project/39f686fe-9e69-4343-a9ce-129e22bff288?utm_source=github&utm_medium=referral&page=upgrade-pr

Snyk has created this PR to upgrade @aws-sdk/client-personalize from 3.363.0 to 3.391.0. See this package in npm: https://www.npmjs.com/package/@aws-sdk/client-personalize See this project in Snyk: https://app.snyk.io/org/datamanagement/project/39f686fe-9e69-4343-a9ce-129e22bff288?utm_source=github&utm_medium=referral&page=upgrade-pr

Snyk has created this PR to upgrade node-fetch from 2.6.9 to 2.6.12. See this package in npm: https://www.npmjs.com/package/node-fetch See this project in Snyk: https://app.snyk.io/org/datamanagement/project/39f686fe-9e69-4343-a9ce-129e22bff288?utm_source=github&utm_medium=referral&page=upgrade-pr

* fix: marketo bulk upload import issue (#2559) * chore(release): 1.40.2 --------- Co-authored-by: shrouti1507 <[email protected]> Co-authored-by: GitHub Actions <[email protected]>

chore: improce error message for HS

…2581) add all traits to profile properties in track call

fix(iterable): added support of id parameter for update events

* migrate test cases to new component test suit * chore: migrate lambda tests * chore: migrate kochava tests * chore: migrate lambda tests * chore: migrate leanplum router tests * chore: migrate lemnisk tests * chore: migrate lytics tests * chore: migrate mailjet tests * chore: migrate mailmodo tests * chore: migrate marketo_static_list tests * chore: migrate moengage tests * chore: migrate moetate tests * chore: migrate ometria tests * chore: migrate one_signal tests * chore: migrate pagerduty tests * chore: migrate pinterest_tag tests * chore: migrate pardot tests

* feat: onboard launchdarkly audience * docs: add comment * test: add testcases * refactor: update identifier logic * feat: add context kind mapping * fix: use removeUndefinedNullEmptyExclBoolInt util * chore: stop sending audience url * chore: remove extra argument in prepareIdentifiersList * fix: endpoint * fix: testcases * feat: add batching support * chore: add unit test cases

* feat(mixpanel): add incremental properties support * docs: add comment * test: update testcase

* chore: update workflow engine npm pkg name * fix: workflow engine version in package-lock.json

while preparing the event for profiles<>redis store

* chore: update redis expiry time * comment addressed * comment addressed

…2575) added dropdown new option to selct datacenter

…and skipped statuses (#2606) fix: marketo response handler

* feat: onboard fullstory cloud mode * test: add more tests for identify

…removed and skipped statuses" (#2619) Revert "fix(INT-339): marketo response handler to handle static list removed and skipped statuses (#2606)" This reverts commit e3fed49.

…nnel webhook (#2563) * fix: slack send event to event specific channel * small fixes * incorporated legacy method * added test cases and small fixes * fixed payload for modern webhooks for not sending app details * feat:added blacklisted event option

#2598) * feat: introduce de-activation of authStatus for access_denied or invalid_grant errors Signed-off-by: Sai Sankeerth <[email protected]> * fix: test-cases-1 Signed-off-by: Sai Sankeerth <[email protected]> * chore: remove all references of disable_dest and remove the const disable_dest Signed-off-by: Sai Sankeerth <[email protected]> * chore: refactor to use util methods for getting auth error category and access-token Signed-off-by: Sai Sankeerth <[email protected]> * fix: pardot router component test failure and remove unnecessary code Signed-off-by: Sai Sankeerth <[email protected]> * fix: remove unused code --------- Signed-off-by: Sai Sankeerth <[email protected]> Co-authored-by: Sai Sankeerth <[email protected]>

…and skipped statuses (#2620) * fix: marketo response handler * small fix * chore: comment addressed

* fix(transformation): block ipv6 requests in user transformer * test cases * block ips starting with localhost octet * assign to variable

codecov · 2023-09-20T06:46:01Z

Codecov Report

Patch coverage: 90.70% and project coverage change: +0.03% 🎉

Comparison is base (ed27a11) 86.85% compared to head (9440e39) 86.89%.
Report is 41 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2633      +/-   ##
===========================================
+ Coverage    86.85%   86.89%   +0.03%     
===========================================
  Files          567      603      +36     
  Lines        27964    28129     +165     
  Branches      6661     6693      +32     
===========================================
+ Hits         24288    24442     +154     
- Misses        3342     3354      +12     
+ Partials       334      333       -1

Files Changed	Coverage Δ
src/adapters/networkhandler/authConstants.js	`100.00% <ø> (ø)`
src/cdk/v2/handler.ts	`94.44% <ø> (ø)`
src/controllers/misc.ts	`20.00% <0.00%> (-13.34%)`	⬇️
...ogle_adwords_offline_conversions/networkHandler.js	`94.28% <ø> (-0.26%)`	⬇️
...ations/google_adwords_offline_conversions/utils.js	`94.66% <ø> (+0.32%)`	⬆️
src/v0/destinations/iterable/transform.js	`95.41% <ø> (ø)`
src/v0/destinations/iterable/util.js	`94.02% <ø> (-0.45%)`	⬇️
src/v0/destinations/klaviyo/util.js	`99.20% <ø> (ø)`
src/v0/destinations/slack/util.js	`92.85% <ø> (ø)`
...st/integrations/destinations/lambda/router/data.ts	`100.00% <ø> (ø)`
... and 65 more

... and 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

sonarqubecloud · 2023-09-20T06:46:29Z

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

krishna2020 · 2023-09-21T06:24:57Z

@Sanjay-Veernala Raise a new PR, this has unwanted changes

snyk-bot and others added 30 commits September 8, 2023 03:18

chore(release): pull hotfix-release/v1.40.2 into main (#2560)

90edb64

* fix: marketo bulk upload import issue (#2559) * chore(release): 1.40.2 --------- Co-authored-by: shrouti1507 <[email protected]> Co-authored-by: GitHub Actions <[email protected]>

chore: improve error message for HS (#2576)

d2619f9

chore: improce error message for HS

feat(tiktok_ads): messageId to event_id mapping support

1e4a606

fix: json paths for non tracks events for warehouse (#2571)

b47219d

fix(bugsnag): alerts

3dce243

feat: add support for updating profile properties through track call (#…

21852d9

…2581) add all traits to profile properties in track call

chore(release): 1.41.0

32856ea

fix(INT-591): added support of id parameter for update events (#2595)

47ab4ba

fix(iterable): added support of id parameter for update events

fix(iterable): updated mappings (#2602)

eec0d63

added profiling endpoints to fetch profiles

afe7231

fix: remove hashing on twclid for twitter ads (#2605)

bc01bdf

chore: fix sonar and lint issues (#2607)

9122dd2

feat(mixpanel): add incremental properties support (#2550)

46c4ccc

* feat(mixpanel): add incremental properties support * docs: add comment * test: update testcase

feat: google ads v14 upgrade (#2578)

735aac1

fix: pyroscope endpoints (#2613)

e1a60f2

chore: run build docker image action on self hosted runners (#2534)

f0fe7eb

chore: update workflow engine npm pkg name (#2610)

b9f0bbc

* chore: update workflow engine npm pkg name * fix: workflow engine version in package-lock.json

fix(redis): add destination_id in the key

8f233c6

while preparing the event for profiles<>redis store

chore(release): 1.41.1

78294ad

chore(INT-631): update redis expiry time (#2609)

28d4021

* chore: update redis expiry time * comment addressed * comment addressed

fix(INT-523): replaced flag with dropdown for datacenter selection (#…

8716fd0

…2575) added dropdown new option to selct datacenter

fix(INT-339): marketo response handler to handle static list removed …

d543f47

…and skipped statuses (#2606) fix: marketo response handler

feat: onboard fullstory cloud mode (#2536)

9babaf8

* feat: onboard fullstory cloud mode * test: add more tests for identify

Revert "fix(INT-339): marketo response handler to handle static list …

c8b3b5a

…removed and skipped statuses" (#2619) Revert "fix(INT-339): marketo response handler to handle static list removed and skipped statuses (#2606)" This reverts commit e3fed49.

anantjain45823 and others added 7 commits September 19, 2023 14:07

fix(INT-339): marketo response handler to handle static list removed …

c49bb10

…and skipped statuses (#2620) * fix: marketo response handler * small fix * chore: comment addressed

fix(transformation): block ipv6 requests in user transformer (#2618)

bf9d038

* fix(transformation): block ipv6 requests in user transformer * test cases * block ips starting with localhost octet * assign to variable

fix: seperate build steps for ruddestack and rudderlabs repo (#2625)

39c3e24

fix(mixpanel): batch event ordering (#2608)

3a79023

fix(snyk): upgraded versiions for aws packages

9440e39

Sanjay-Veernala requested review from a team and sivashanmukh as code owners September 20, 2023 06:37

Sanjay-Veernala requested review from ujjwal-ab and ItsSudip September 20, 2023 06:37

krishna2020 closed this Sep 21, 2023

devops-github-rudderstack deleted the feat/snyk-fixes-test branch December 20, 2023 01:16

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat/snyk fixes test #2633

Feat/snyk fixes test #2633

Sanjay-Veernala commented Sep 20, 2023

codecov bot commented Sep 20, 2023

sonarqubecloud bot commented Sep 20, 2023

krishna2020 commented Sep 21, 2023

Feat/snyk fixes test #2633

Feat/snyk fixes test #2633

Conversation

Sanjay-Veernala commented Sep 20, 2023

Description of the change

Type of change

Related issues

Checklists

Development

Code review

codecov bot commented Sep 20, 2023

Codecov Report

sonarqubecloud bot commented Sep 20, 2023

krishna2020 commented Sep 21, 2023