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

Can owlbot-java use the formatter defined in pom.xml? #1502

Closed
suztomo opened this issue Jul 19, 2022 · 2 comments · Fixed by #1511
Closed

Can owlbot-java use the formatter defined in pom.xml? #1502

suztomo opened this issue Jul 19, 2022 · 2 comments · Fixed by #1511
Assignees
Labels
type: question Request for information or clarification. Not an issue.

Comments

@suztomo
Copy link
Member

suztomo commented Jul 19, 2022

We can eliminate the discrepancy between the two if owlbot-java uses the formatter defined in the project:

@suztomo suztomo self-assigned this Jul 19, 2022
@tmatsuo tmatsuo added the type: question Request for information or clarification. Not an issue. label Jul 20, 2022
@chingor13
Copy link
Contributor

We do not want to run any mvn commands in the post-processor that use code from the repository. owl-bot runs in a privileged (albeit limited) environment and you can craft a pom.xml such that it loads malicious plugins that run arbitrary code.

For this reason, we included the java formatter jar in the post-processor image and ran it directly.

@suztomo
Copy link
Member Author

suztomo commented Jul 27, 2022

@chingor13 Thank you for the background. That's a good principle worth written down somewhere (my todo).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: question Request for information or clarification. Not an issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants