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

fix(orchestrator): repair sql injection - application_id #6050

Closed
wants to merge 3 commits into from
Closed

fix(orchestrator): repair sql injection - application_id #6050

wants to merge 3 commits into from

Conversation

jing-zhi
Copy link
Contributor

What this PR does / why we need it:

ChangeLog

fix(instance.go):repair sql injection - application_id(修复sql注入-application_id 问题)

Language Changelog
🇺🇸 English repair sql injection - application_id
🇨🇳 中文 修复sql注入-application_id 问题

@chengjoey
Copy link
Contributor

looks good~
there are some parts of this PR that are not very standard like fix title, test coverage.
I will take a look tomorrow

@jing-zhi jing-zhi changed the title fix(instance.go):repair sql injection - application_id fix(orchestrator):repair sql injection - application_id Sep 22, 2023
@erda-bot erda-bot added the orchestrator workload management label Sep 22, 2023
@jing-zhi jing-zhi changed the title fix(orchestrator):repair sql injection - application_id fix(orchestrator): repair sql injection - application_id Sep 22, 2023
@jing-zhi jing-zhi closed this by deleting the head repository Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix orchestrator workload management
Development

Successfully merging this pull request may close these issues.

3 participants