Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat!: fx and interscheme implementation #492

Merged
merged 30 commits into from
Dec 11, 2024
Merged

feat!: fx and interscheme implementation #492

merged 30 commits into from
Dec 11, 2024

Conversation

vijayg10
Copy link
Contributor

@vijayg10 vijayg10 commented Aug 29, 2024

Changes:

  • Currency Conversion (FX)
    • There are no changes required in the ALS implementation related to FX
  • Interscheme
    • Proxy cache implementation using redis
    • Parameterize switch id

oderayi and others added 18 commits June 25, 2024 10:01
* fix(csi-241): fixed unit-tests

* fix(csi-241): fixed unit-tests

* fix(csi-241): fixed unit-tests

* fix(csi-241): fixed unit-tests

* test: temporarily disable int test shutdown

* fix: update parameters for sendRequest and update tests

* chore: update deps, re-enable int test shutdown

---------

Co-authored-by: Steven Oderayi <[email protected]>
* fix(csi-241): fixed unit-tests

* fix(csi-241): fixed unit-tests

* fix(csi-241): fixed unit-tests

* fix(csi-241): fixed unit-tests

* feat(csi-16): refactoring - moved getPartiesByTypeAndID to a separate file

* ci: fix docker build

* feat(csi-263): implemented getPartiesByTypeAndID proxy changes (#485)

* feat(csi-263): implemented getPartiesByTypeAndID proxy changes

* feat(csi-263): implemented getPartiesByTypeAndID proxy changes

* feat(csi-263): update npm test:... scripts

* feat(csi-263): added networks to redis in d-c

* feat(csi-263): added proxyCache logic for success PUT /parties

* feat(csi-263): added proxyCache logic for error PUT /parties

* feat(csi-263): updated proxyConfig format

* feat(csi-263): added jest-unit.config.js

* feat(csi-215): updated test-harness version to use fx

* chore(snapshot): 15.4.0-snapshot.0

* chore(snapshot): 15.4.0-snapshot.1

* chore(snapshot): 15.4.0-snapshot.2

* chore(snapshot): 15.4.0-snapshot.3

* chore(snapshot): 15.4.0-snapshot.4

* chore(snapshot): 15.4.0-snapshot.5

* chore(snapshot): 15.4.0-snapshot.6

* chore(snapshot): 15.4.0-snapshot.7

* chore(snapshot): 15.4.0-snapshot.8

* chore(snapshot): 15.4.0-snapshot.9

* chore(snapshot): 15.4.0-snapshot.10

* chore(snapshot): 15.4.0-snapshot.11

* feat: make proxy routing optional & update health check (#486)

* chore(csi-16): updated central-services-shared version

* chore(snapshot): 15.4.0-snapshot.12

* chore(snapshot): 15.4.0-snapshot.13

* chore(snapshot): 15.4.0-snapshot.14

* chore(csi-16): inited proxies cache

* chore(snapshot): 15.4.0-snapshot.15

* chore(snapshot): 15.4.0-snapshot.16

* feat(csi-263): added proxy int-test from parties handler

* feat(csi-263): added proxy int-test from parties handler

* feat(csi-263): added --runInBand to unit tests

* feat(csi-16): proxyCacheTtlSec = 40

* chore(snapshot): 15.4.0-snapshot.17

* feat(csi-16): proxyCacheTtlSec = 40

* feat(csi-16): proxyCacheTtlSec = 40

* chore(snapshot): 15.4.0-snapshot.18

* Update docker/account-lookup-service/default.json

Co-authored-by: Steven Oderayi <[email protected]>

* Update test/integration-config.json

Co-authored-by: Steven Oderayi <[email protected]>

---------

Co-authored-by: Steven Oderayi <[email protected]>

* feat(csi-334): e2e als-proxy flow validation (#487)

* feat(csi-263): implemented getPartiesByTypeAndID proxy changes

* feat(csi-263): implemented getPartiesByTypeAndID proxy changes

* feat(csi-263): update npm test:... scripts

* feat(csi-263): added networks to redis in d-c

* feat(csi-263): added proxyCache logic for success PUT /parties

* feat(csi-263): added proxyCache logic for error PUT /parties

* feat(csi-263): updated proxyConfig format

* feat(csi-263): added jest-unit.config.js

* feat(csi-215): updated test-harness version to use fx

* chore(snapshot): 15.4.0-snapshot.0

* chore(snapshot): 15.4.0-snapshot.1

* chore(snapshot): 15.4.0-snapshot.2

* chore(snapshot): 15.4.0-snapshot.3

* chore(snapshot): 15.4.0-snapshot.4

* chore(snapshot): 15.4.0-snapshot.5

* chore(snapshot): 15.4.0-snapshot.6

* chore(snapshot): 15.4.0-snapshot.7

* chore(snapshot): 15.4.0-snapshot.8

* chore(snapshot): 15.4.0-snapshot.9

* chore(snapshot): 15.4.0-snapshot.10

* chore(snapshot): 15.4.0-snapshot.11

* feat: make proxy routing optional & update health check (#486)

* chore(csi-16): updated central-services-shared version

* chore(snapshot): 15.4.0-snapshot.12

* chore(snapshot): 15.4.0-snapshot.13

* chore(snapshot): 15.4.0-snapshot.14

* chore(csi-16): inited proxies cache

* chore(snapshot): 15.4.0-snapshot.15

* chore(snapshot): 15.4.0-snapshot.16

* feat(csi-263): added proxy int-test from parties handler

* feat(csi-263): added proxy int-test from parties handler

* feat(csi-263): added --runInBand to unit tests

* feat(csi-16): proxyCacheTtlSec = 40

* chore(snapshot): 15.4.0-snapshot.17

* feat(csi-16): proxyCacheTtlSec = 40

* feat(csi-16): proxyCacheTtlSec = 40

* chore(snapshot): 15.4.0-snapshot.18

* Update docker/account-lookup-service/default.json

Co-authored-by: Steven Oderayi <[email protected]>

* Update test/integration-config.json

Co-authored-by: Steven Oderayi <[email protected]>

* feat(csi-334): added source to proxyCache for payee case

* chore(snapshot): 15.4.0-snapshot.19

* feat(csi-334): added axios-retry

* chore(snapshot): 15.4.0-snapshot.20

* fix(csi-321): fixed failed GP tests

* chore(snapshot): 15.4.0-snapshot.21

* fix(csi-321): fixed case of sendCallback to requester

* chore(snapshot): 15.4.0-snapshot.22

* fix(csi-321): added contextLogger

* fix(csi-334): add proxyLib with redis-cluster

* chore(snapshot): 15.4.0-snapshot.23

* fix(csi-334): add proxyLib with redis-cluster

* chore(snapshot): 15.4.0-snapshot.24

* fix(csi-334): fixed config

* chore(snapshot): 15.4.0-snapshot.25

* fix(csi-334): used new proxyLib

---------

Co-authored-by: Steven Oderayi <[email protected]>

* fix(csi-334): switched to redis standalone; improved logging; added int-test

* fix(csi-334): improved logging

* chore(snapshot): 15.4.0-snapshot.26

* fix(csi-418): fixed oracle update logic; added tests (#490)

* fix(csi-418): fixed oracle update logic; added tests

* chore(snapshot): 15.4.0-snapshot.27

* fix(csi-418): fixed oracle update logic; added tests

* chore(snapshot): 15.4.0-snapshot.28

* feat: PROXY_MAP

* chore(snapshot): 15.4.0-snapshot.29

* feat(mojaloop/#4000): timeout handler (#489)

---------

Co-authored-by: Steven Oderayi <[email protected]>
Co-authored-by: Kalin Krustev <[email protected]>
@kleyow kleyow changed the title feat: fx and interscheme implementation feat!: fx and interscheme implementation Sep 26, 2024
Dockerfile Fixed Show resolved Hide resolved
Copy link

sonarqubecloud bot commented Oct 21, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
13.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

shashi165
shashi165 previously approved these changes Nov 11, 2024
package.json Outdated Show resolved Hide resolved
@elnyry-sam-k
Copy link
Member

Thanks for the PR and initial review Shashi..

Some comments:

  1. About "Mock proxy", can you please add some text / documentation to describe it and the usage? Also the version is 0.x
  2. Time-out handler - please document or provide links if it exists elsewhere
  3. https://github.com/mojaloop/inter-scheme-proxy-cache-lib still shows as EXPERIMENTAL and doesn't have an official release yet - needs to be addressed before we merge this to main
  4. Are there sequence diagrams for the new changes somewhere? Please confirm

Thank you!

@kalinkrustev
Copy link
Contributor

@geka-evk can you check the comments above about the proxy

@geka-evk
Copy link
Contributor

geka-evk commented Dec 9, 2024

  1. Added "Mock proxy" README
  2. Added ALS Timeout flow description
  3. inter-scheme-proxy-cache-lib has version 2.3.0.
  4. Added links to Discovery flow diagram and Proxy implementation details into README documentation

docs/Proxy/P2P.md Outdated Show resolved Hide resolved
elnyry-sam-k
elnyry-sam-k previously approved these changes Dec 10, 2024
Copy link
Member

@elnyry-sam-k elnyry-sam-k left a comment

Choose a reason for hiding this comment

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

Thanks Eugen

@elnyry-sam-k
Copy link
Member

Thanks @geka-evk , approved the PR but inter-scheme-proxy-cache-lib needs to have "releases" on github repos similar to other mojaloop repos, not just tags.

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
13.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@geka-evk geka-evk merged commit c44e5e0 into main Dec 11, 2024
15 of 16 checks passed
@geka-evk geka-evk deleted the feat/fx-impl branch December 11, 2024 11:31
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.

6 participants