Prepare v0.20.5 release #2448
Merged
Prepare v0.20.5 release #2448
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
As part of this change, also update the format package to unmangle the variables slightly differently--just remove the wildcard prefix instead of translating the variable names. This makes it easier to tell where the variables came from in the first place and is a bit less complicated. Fixes open-policy-agent#2439 Signed-off-by: Torin Sandall <torinsandall@gmail.com> (cherry picked from commit fe18f11)
The current Wasm import table was missing one required function, `opa_println`. This just adds it to the list with a quick description. Signed-off-by: Patrick East <east.patrick@gmail.com> (cherry picked from commit 3eea0e8)
Previously it was being bound to `$result` but this was then stripped out of the resultset by the planner. To avoid this issue we need the variable to not be seen as a wildcard or generated var. The new one is just `result`. The documentation is also now updated to show this behavior. The various client SDK's can strip it out as needed. The idea is that this is going to just be a part of the built WASM binary format. Anyone building with the lower level API's using ad-hoc queries will not need to worry about anything changing. Fixes: open-policy-agent#2441 Signed-off-by: Patrick East <east.patrick@gmail.com> (cherry picked from commit 729a853)
Signed-off-by: Patrick East <east.patrick@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Including a handful of fixes for:
opa buildgenerated WASM binary always returns empty set #2441To unblock some cases of the
opa buildcommand and WASM users.