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

TCK validator BKM typeRef #468

Merged
merged 4 commits into from
Oct 14, 2021
Merged

Conversation

tarilabs
Copy link
Member

@tarilabs tarilabs commented Sep 5, 2021

Follow-up of:

for the BKM's variable/encapsulatedLogic typeRef.

As discussed in the meeting, we can at least detect when it's clearly not a function, and we were expecting a function type to be the in the BKM's variable/encapsulatedLogic typeRef --the return type is expected in the "body" encapsulatedLogic.expression typeRef

Please ignore 9999-dummy-test-01.xml which has no functional scope in this PR.

This also fixes a potential XXE vulnerability

@tarilabs
Copy link
Member Author

tarilabs commented Sep 5, 2021

Demonstrating the GHA runner failure for the TCK Validator

https://github.com/dmn-tck/tck/pull/468/checks?check_run_id=3517372502

image

@tarilabs
Copy link
Member Author

tarilabs commented Sep 5, 2021

Commit demo2 f8d7dc9
demonstrates it passing in other cases: https://github.com/dmn-tck/tck/pull/468/checks?check_run_id=3517380385

image

@tarilabs tarilabs marked this pull request as ready for review September 5, 2021 10:56
@tarilabs tarilabs added this to the 2021.next milestone Oct 12, 2021
@opatrascoiu
Copy link
Contributor

Looks good to me. Thanks @tarilabs

@tarilabs
Copy link
Member Author

discussed in meeting TCK 2021-10-14, merging.

@tarilabs tarilabs merged commit 7278381 into dmn-tck:master Oct 14, 2021
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