-
Notifications
You must be signed in to change notification settings - Fork 187
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
Flar UI vue 3 sso #589
base: main
Are you sure you want to change the base?
Flar UI vue 3 sso #589
Conversation
* added jenkinsfile * updated changes to ignore errcheck in linting, included build.sh * added runtests.sh * added runtests.sh * updated jenkins file * updated jenkins file
if (typeof window !== "undefined") { | ||
let uuid = localStorage.getItem("uuid") | ||
if (!uuid) { | ||
uuid = generateUUID() |
Check failure
Code scanning / CodeQL
Insecure randomness High
Math.random()
} | ||
|
||
filePath := "./browser/flagr-ui/dist/" + r.URL.Path | ||
if _, err := os.Stat(filePath); err == nil && filepath.Ext(r.URL.Path) != "" { |
Check failure
Code scanning / CodeQL
Uncontrolled data used in path expression High
user-provided value
|
||
filePath := "./browser/flagr-ui/dist/" + r.URL.Path | ||
if _, err := os.Stat(filePath); err == nil && filepath.Ext(r.URL.Path) != "" { | ||
http.ServeFile(w, r, filePath) // Serve the static file directly |
Check failure
Code scanning / CodeQL
Uncontrolled data used in path expression High
user-provided value
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #589 +/- ##
==========================================
- Coverage 81.19% 80.64% -0.56%
==========================================
Files 28 29 +1
Lines 2271 2428 +157
==========================================
+ Hits 1844 1958 +114
- Misses 337 368 +31
- Partials 90 102 +12 ☔ View full report in Codecov by Sentry. |
@@ -17,7 +17,7 @@ jobs: | |||
- name: Publish Docker SemVer Tag | |||
uses: elgohr/Publish-Docker-Github-Action@v5 | |||
with: | |||
name: openflagr/flagr | |||
name: Allen-Career-Institute/flagr |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
quick question, is this supposed to be a PR for openflagr upstream or your own github org's fork?
Description
Motivation and Context
How Has This Been Tested?
Types of changes
Checklist: