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.

Sign up for GitHub

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

Jump to bottom

Prevent bugs caused by use of unsafe APIs #91

Open
britter opened this issue Feb 7, 2023 · 4 comments · May be fixed by #95
Open

Prevent bugs caused by use of unsafe APIs #91

britter opened this issue Feb 7, 2023 · 4 comments · May be fixed by #95
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@britter
Copy link
Member

britter commented Feb 7, 2023 •
edited
Loading

PS. https://github.com/vlsi/github-actions-random-matrix and/or https://github.com/policeman-tools/forbidden-apis/wiki/BundledSignatures could help you capture such bugs in the future

Originally posted by @vlsi in #87 (comment)
@britter britter added enhancement New feature or request help wanted Extra attention is needed needs-decision Issues marked for decision by the team whether or not they will be implemented labels Feb 7, 2023
@vlsi
Copy link

vlsi commented Feb 7, 2023

I can help you with integrating the random ci / forbiddenapis

@britter
Copy link
Member Author

britter commented Feb 8, 2023

@vlsi great! I'd like to discuss it at the next team meeting this Friday and will ping you about the result of the discussion.

@britter
Copy link
Member Author

britter commented Feb 10, 2023

Team decision

We'd like to give https://github.com/policeman-tools/forbidden-apis/wiki/BundledSignatures a trye.

@vlsi would you like to create a PR for integrating it into the build?

@britter britter removed the needs-decision Issues marked for decision by the team whether or not they will be implemented label Feb 10, 2023
@vlsi
Copy link

vlsi commented Feb 10, 2023

Any conclusions on the randomized matrix?
See https://twitter.com/VladimirSitnikv/status/1623265219383263232

@vlsi vlsi linked a pull request Feb 10, 2023 that will close this issue
Open
vlsi added a commit to vlsi/build-parameters that referenced this issue Feb 10, 2023
@vlsi
Add forbidden-apis check to prevent accidental use of locale-dependen…
2a294b9 
…t APIs

See https://github.com/policeman-tools/forbidden-apis/wiki/BundledSignatures
See https://github.com/policeman-tools/forbidden-apis/wiki/GradleUsage

Fixes gradlex-org#91
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add forbidden-apis check to prevent accidental use of locale-dependent APIs vlsi/build-parameters
2 participants
@vlsi @britter