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

ED-162: +payment token bouncer data #713

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dinama
Copy link
Contributor

@dinama dinama commented Sep 4, 2021

No description provided.

@dinama dinama requested review from a team as code owners September 4, 2021 18:10
Copy link
Contributor

@keynslug keynslug left a comment

Choose a reason for hiding this comment

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

Насколько я помню, мы собирались запекание bouncer ctx в токен отложить до тех времён, когда начнём выписывать платёжные токены через token keeper. С этим планом что-то изменилось?

@dinama
Copy link
Contributor Author

dinama commented Nov 1, 2021

Насколько я помню, мы собирались запекание bouncer ctx в токен отложить до тех времён, когда начнём выписывать платёжные токены через token keeper. С этим планом что-то изменилось?

просто этот PR висел на мне и надо было что-то сделать. я помню, что ты хотел в токен класть BouncerContext - я положил. хотя как считал так и считаю это угловатым решением. независимо от того в каком модуле будет функция сериализации .

@dinama
Copy link
Contributor Author

dinama commented Nov 2, 2021

@keynslug

подумал, что можно попробовать сделать так

  • в сapi в create*AccessToken создать не фрагмент, а полноценны BouncerContext на основе прототипа+build
  • класть его в метаданные AccessToken
  • в pcidss сreatePaymentResource перекладывать из метаданных AccessToken в метаданные PaymentResource
  • в capi снова из метаданных PaymentResource объединять с контекстам созданным из прототипов используя наш контекст вместо new для Acc

тогда, возможно, получится не так криво, и будет повторное использование capi_bouncer_context+buil+prototype
правда. не знаю когда теперь смогу это сделать

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.

2 participants