Skip to content

replace github.event expressions with env variables in telegram-bot workflow#856

Closed
dumitory-dev wants to merge 1 commit into
Samsung:masterfrom
dumitory-dev:refactor/ci-telegram-bot-workflow
Closed

replace github.event expressions with env variables in telegram-bot workflow#856
dumitory-dev wants to merge 1 commit into
Samsung:masterfrom
dumitory-dev:refactor/ci-telegram-bot-workflow

Conversation

@dumitory-dev
Copy link
Copy Markdown
Collaborator

Description

Replace direct ${{ github.event.* }} expressions in the run block with
environment variables to prevent potential script injection.

How Has This Been Tested?

If your pull request does not include new tests for this change, please describe:

  • How to reproduce unwanted behaviour without this fix
  • How to make sure your change fixes this unwanted behaviour
  • What environment have you been using for your tests

Checklist:

  • My code follows the PEP 8
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@dumitory-dev
Copy link
Copy Markdown
Collaborator Author

test

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.

1 participant