Skip to content

[CALCITE-7165] Update OWASP plugin from 10.0.4 to 12.13 in Avatica#287

Closed
stoty wants to merge 1 commit intoapache:mainfrom
stoty:CALCITE-7165
Closed

[CALCITE-7165] Update OWASP plugin from 10.0.4 to 12.13 in Avatica#287
stoty wants to merge 1 commit intoapache:mainfrom
stoty:CALCITE-7165

Conversation

@stoty
Copy link
Copy Markdown
Contributor

@stoty stoty commented Sep 9, 2025

No description provided.

@zabetak
Copy link
Copy Markdown
Member

zabetak commented Sep 9, 2025

In Calcite build scripts we have some conditional parts based on the JavaVersion so maybe you can take some inspiration from there for making the plugin conditional.

@stoty
Copy link
Copy Markdown
Contributor Author

stoty commented Sep 10, 2025

Thanks @zabetak .
Unfortunately that solution doesn't work, because the build breaks as soon as the wrong version plugin is imported, gradle never gets as far as applying any configuration.

@zabetak
Copy link
Copy Markdown
Member

zabetak commented Sep 10, 2025

@stoty Please check #294 and let me know what you think.

@stoty
Copy link
Copy Markdown
Contributor Author

stoty commented Sep 17, 2025

Other PR merged for this issue

@stoty stoty closed this Sep 17, 2025
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