Skip to content

Conversation

@shaypal5
Copy link
Member

@shaypal5 shaypal5 commented Feb 14, 2024

Hey, I'd love your help, @Borda .

I've set up a new dedicated Github Actions environment for this repository, named pypi_publish, which has a PYPI_PASSWORD secret with cachier's PyPI API token.

However, from what I understand, a Github Actions environment is a job-level property:
https://docs.github.com/en/actions/deployment/targeting-different-environments/using-environments-for-deployment#using-an-environment

This seems a bit risky to me; a possible attack on PyPI secrets is now not only doing something fishy in the gh-action-pypi-publish@v1.8.11 action, but also in any unrelated action we use earlier in the flow (in this case, AButler/upload-release-assets@v3.0).

So my question is, can you figure out a way to separate the PyPI upload step into a separate job in the same flow? Alternatively, is there a way to set up the environment for only a single step?

Cheers!

@shaypal5 shaypal5 requested a review from Borda February 14, 2024 20:13
@shaypal5
Copy link
Member Author

Awaiting your approval, @Borda

@shaypal5 shaypal5 merged commit f0cc950 into master Feb 15, 2024
@shaypal5 shaypal5 deleted the release_flow_fix branch February 15, 2024 06:37
Copy link
Contributor

@Borda Borda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants