-
Notifications
You must be signed in to change notification settings - Fork 5
Los app #6543
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
base: master
Are you sure you want to change the base?
Conversation
|
msw: { | ||
handlers: [ | ||
http.get(LosUrl.KODEVERK_LOS, () => HttpResponse.json(alleKodeverkLos)), | ||
http.get(LosUrl.HENT_OPPGAVER_FOR_AVDELING.replace('ø', '%C3%B8'), () => |
Check failure
Code scanning / CodeQL
Incomplete string escaping or encoding High
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 5 days ago
To fix the problem, we should ensure that all occurrences of 'ø' in the string are replaced with '%C3%B8', not just the first. The best way to do this is to use a regular expression with the global (g
) flag: .replace(/ø/g, '%C3%B8')
. This change should be made directly on line 57, replacing the current .replace('ø', '%C3%B8')
with .replace(/ø/g, '%C3%B8')
. No additional imports or definitions are needed, as this is standard JavaScript functionality.
-
Copy modified line R57
@@ -54,7 +54,7 @@ | ||
msw: { | ||
handlers: [ | ||
http.get(LosUrl.KODEVERK_LOS, () => HttpResponse.json(alleKodeverkLos)), | ||
http.get(LosUrl.HENT_OPPGAVER_FOR_AVDELING.replace('ø', '%C3%B8'), () => | ||
http.get(LosUrl.HENT_OPPGAVER_FOR_AVDELING.replace(/ø/g, '%C3%B8'), () => | ||
HttpResponse.json(OPPGAVER_FOR_AVDELING), | ||
), | ||
], |
msw: { | ||
handlers: [ | ||
http.get(LosUrl.KODEVERK_LOS, () => HttpResponse.json(alleKodeverkLos)), | ||
http.get(LosUrl.HENT_OPPGAVER_PER_DATO.replace('ø', '%C3%B8'), () => HttpResponse.json(OPPGAVER_PER_DATO)), |
Check failure
Code scanning / CodeQL
Incomplete string escaping or encoding High
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 5 days ago
The best way to fix this problem is to ensure that all occurrences of 'ø'
in the string are replaced, not just the first. This can be done by using a regular expression with the global flag: .replace(/ø/g, '%C3%B8')
. However, since the intention is to encode a URL, the most robust and future-proof solution is to use the built-in encodeURI
or encodeURIComponent
functions, which handle all necessary character encodings for URLs. In this context, if the entire URL needs to be encoded, use encodeURI
; if only a component (such as a path segment or query parameter), use encodeURIComponent
. Since the code is replacing 'ø'
with its percent-encoded form, it is likely that the intention is to encode the URL. Therefore, replace .replace('ø', '%C3%B8')
with encodeURI(LosUrl.HENT_OPPGAVER_PER_DATO)
. This change should be made on line 65 of apps/los-avdelingsleder/src/nokkeltall/tilBehandling/TilBehandlingPanel.stories.tsx
. No additional imports are needed, as encodeURI
is a global JavaScript function.
-
Copy modified line R65
@@ -62,7 +62,7 @@ | ||
msw: { | ||
handlers: [ | ||
http.get(LosUrl.KODEVERK_LOS, () => HttpResponse.json(alleKodeverkLos)), | ||
http.get(LosUrl.HENT_OPPGAVER_PER_DATO.replace('ø', '%C3%B8'), () => HttpResponse.json(OPPGAVER_PER_DATO)), | ||
http.get(encodeURI(LosUrl.HENT_OPPGAVER_PER_DATO), () => HttpResponse.json(OPPGAVER_PER_DATO)), | ||
], | ||
}, | ||
}, |
No description provided.