Skip to content

Add awesome new feature#2

Open
christophelevis wants to merge 1 commit into
mainfrom
add-feature
Open

Add awesome new feature#2
christophelevis wants to merge 1 commit into
mainfrom
add-feature

Conversation

@christophelevis

Copy link
Copy Markdown
Owner

No description provided.

@sonarqubecloud

sonarqubecloud Bot commented Sep 15, 2023

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions

63.0% Coverage on New Code (required ≥ 80%)
E Reliability Rating on New Code (required ≥ A)
E Security Rating on New Code (required ≥ A)

See analysis details on SonarCloud

idea Catch issues before they fail your Quality Gate with our IDE extension SonarLint SonarLint

{% block content %}
<div class="full-page d-flex flex-wrap justify-content-center align-items-center">
<div class="container d-flex flex-column flex-wrap justify-content-start align-items-center text-center py-5 m-auto margin-mobile">
<h1>{{ name }} # {% autoescape false %} {{ pokemon_id }} {% endautoescape %}</h1>

Check failure

Code scanning / SonarCloud

Endpoints should not be vulnerable to reflected cross-site scripting (XSS) attacks

<!--SONAR_ISSUE_KEY:AYqaLQLBRmNmDrnehO-m-->Change this code to not reflect user-controlled data. <p>See more on <a href="https://sonarcloud.io/project/issues?id=christophelevis_chris-python-flask-demo&issues=AYqaLQLBRmNmDrnehO-m&open=AYqaLQLBRmNmDrnehO-m&pullRequest=2">SonarCloud</a></p>
Comment thread pokedex/helper.py

def get_single_pokemon(self, pokemon_id):
statement = f"SELECT * FROM POKEDEX WHERE id = '{pokemon_id}'"
return self.__conn.execute(statement).fetchone()

Check failure

Code scanning / SonarCloud

Database queries should not be vulnerable to injection attacks

<!--SONAR_ISSUE_KEY:AYqaLQPpRmNmDrnehO-r-->Change this code to not construct SQL queries directly from user-controlled data. <p>See more on <a href="https://sonarcloud.io/project/issues?id=christophelevis_chris-python-flask-demo&issues=AYqaLQPpRmNmDrnehO-r&open=AYqaLQPpRmNmDrnehO-r&pullRequest=2">SonarCloud</a></p>
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.

3 participants