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

Changes required for upgrading CQF Ruler #20

Open
aeyates opened this issue Dec 12, 2022 · 1 comment
Open

Changes required for upgrading CQF Ruler #20

aeyates opened this issue Dec 12, 2022 · 1 comment
Milestone

Comments

@aeyates
Copy link
Collaborator

aeyates commented Dec 12, 2022

Two changes are needed to the hook request to work with CQF Ruler 0.5.1 and above.

  1. The property "applyCql" is no longer valid. It can be removed at any time, as its removal doesn't break 0.5.0.
  2. The syntax for fhirAuthorization has changed. I have logged this as a bug with CQF Ruler. But in the event that it isn't fixed soon, the workaround will be to modify hookRequest.mustache like this:
"access_token": "Bearer {{bearerToken}}",
"token_type": "Authorization",

Note that this would be a breaking change for 0.5.0, and it does not meet the CDS Hooks specification, so we should only make this change if we need to upgrade before a fix is in place.

@aeyates
Copy link
Collaborator Author

aeyates commented Dec 14, 2022

I went ahead and committed the first change and tested in dev. I've also updated the CQF Ruler ticket with a potential fix.

@aeyates aeyates added this to the Backlog milestone Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants