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

Added configurable port through env 'FLASK_PORT' #23

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

DevenMarrero
Copy link
Contributor

#22

@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 8, 2021 22:38 Inactive
@DevenMarrero DevenMarrero linked an issue Jul 8, 2021 that may be closed by this pull request
varun-patel
varun-patel previously approved these changes Jul 8, 2021
Copy link
Member

@varun-patel varun-patel left a comment

Choose a reason for hiding this comment

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

thanks for fixing that quickly.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Scan Summary

Tool Critical High Medium Low Status
Security Audit for Infrastructure 0 0 1 1
Python Security Analysis 0 0 2 4
Python Source Analyzer 0 0 0 0

Recommendation

Please review the findings from Code scanning alerts before approving this pull request. You can also configure the build rules or add suppressions to customize this bot 👍

@DevenMarrero DevenMarrero self-assigned this Jul 8, 2021
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 8, 2021 22:48 Inactive
@varun-patel varun-patel self-assigned this Jul 8, 2021
@varun-patel
Copy link
Member

do not merge, work will be done on ci/cd with this branch.

@varun-patel varun-patel added the DO NOT MERGE Maintainers have advised not to merge the linked pull request. label Jul 8, 2021
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 8, 2021 23:01 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 8, 2021 23:06 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 8, 2021 23:27 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 8, 2021 23:41 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 8, 2021 23:48 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 9, 2021 01:34 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 9, 2021 01:45 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 9, 2021 02:07 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 9, 2021 02:12 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 9, 2021 03:40 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 9, 2021 03:50 Inactive
@varun-patel varun-patel temporarily deployed to aristotleapi-pr-23 July 12, 2021 23:37 Inactive
Co-authored-by: Varun Patel <varun@varunpatel.ca>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE Maintainers have advised not to merge the linked pull request. security findings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flask is running in debug mode Implement Configurable Port
3 participants