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

Forking a flow after multiple refreshes throws an NPE #211

Open
jjg-123 opened this issue Oct 16, 2024 · 0 comments
Open

Forking a flow after multiple refreshes throws an NPE #211

jjg-123 opened this issue Oct 16, 2024 · 0 comments

Comments

@jjg-123
Copy link
Contributor

jjg-123 commented Oct 16, 2024

An implicit assumption is that forking the flow happens right after the initial provisioning. If there are several refreshes and exchanges done, resolving the state does not work because "the wrong token" is expected and an NPE is raised. That is too constrictive for sure. Also, a better error should be sent, never an NPE!

This is a critical feature to fix since vault or htgettoken automatically issue refreshes and exchanges initially and users who want to fork flows have no control over this.

@jjg-123 jjg-123 changed the title Forking a flow oafter multiple refreshes throws an NPE Forking a flow after multiple refreshes throws an NPE Oct 19, 2024
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

No branches or pull requests

1 participant