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

Feat/265 url param instantiation #1068

Open
wants to merge 34 commits into
base: main
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
75a82a2
added endpoint to set session values in HomeController
adamhaeger Jan 23, 2025
0eb977d
calculating fields and datamodel in before sessions torage
adamhaeger Jan 24, 2025
0364dd7
Merge branch 'main' into feat/265-url-param-instantiation
adamhaeger Jan 24, 2025
030a76b
wip
adamhaeger Jan 28, 2025
066cd97
clean
adamhaeger Jan 28, 2025
c272983
added null checks
adamhaeger Jan 28, 2025
d0128e4
wip
adamhaeger Jan 28, 2025
0e80736
added null checks
adamhaeger Jan 30, 2025
44a457a
updated swagger
adamhaeger Jan 30, 2025
9b3918a
Merge branch 'main' into feat/265-url-param-instantiation
adamhaeger Jan 30, 2025
c937714
not including query param route in swagger spec as it broke swagger test
adamhaeger Jan 30, 2025
35c31f9
url encoding app and org in js
adamhaeger Jan 30, 2025
0cbb37e
clean
adamhaeger Jan 30, 2025
6b7696a
Added url encoding and CSP with nonce for inline scripts
adamhaeger Jan 30, 2025
b8f75a2
added appId to know mix prefill data from different apps from same se…
adamhaeger Jan 30, 2025
e45a2a7
Added expiry
adamhaeger Jan 31, 2025
7efccaa
checking if we are on a statless app, renamed QueryParams to QueryPar…
adamhaeger Feb 3, 2025
b4de378
handling prefill from query params, added bad request returns in home…
adamhaeger Feb 3, 2025
3d17ad6
implemented IValidateQueryParamPrefill.cs to allow service owners to …
adamhaeger Feb 4, 2025
9299e10
added comments
adamhaeger Feb 4, 2025
385b57e
clean
adamhaeger Feb 4, 2025
c9d7206
wip
adamhaeger Feb 4, 2025
515f1ee
Merge branch 'main' into feat/265-url-param-instantiation
adamhaeger Feb 4, 2025
6b6a418
updated swagger
adamhaeger Feb 4, 2025
35dc6b9
added tests for HomeController
adamhaeger Feb 5, 2025
a6992e4
clean
adamhaeger Feb 5, 2025
2f15ae9
improved AppResourcesMock.cs to make all tests run
adamhaeger Feb 6, 2025
48511e7
cleanup
adamhaeger Feb 6, 2025
b819926
fixed code analysis errors
adamhaeger Feb 6, 2025
6950e77
clean
adamhaeger Feb 6, 2025
0752b4d
merge
adamhaeger Feb 6, 2025
5a9fb8f
Merge branch 'main' into feat/265-url-param-instantiation
adamhaeger Feb 10, 2025
c00b804
removed unsafe json escaping
adamhaeger Feb 11, 2025
9e900a1
clean
adamhaeger Feb 11, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
not including query param route in swagger spec as it broke swagger test
  • Loading branch information
adamhaeger committed Jan 30, 2025
commit c93771433a3637fc8c82e779fd614b0daa095080
2 changes: 1 addition & 1 deletion src/Altinn.App.Api/Controllers/HomeController.cs
Original file line number Diff line number Diff line change
@@ -115,7 +115,7 @@
/// <param name="app"></param>
/// <returns>An HTML file with a small javascript that will set session variables in frontend and redirect to the app.</returns>
[HttpGet]
[Produces("text/html")]
[ApiExplorerSettings(IgnoreApi = true)]
[Route("{org}/{app}/set-query-params")]
public async Task<IActionResult> SetQueryParams(string org, string app)
{
@@ -198,7 +198,7 @@
</body>
</html>";

return Content(htmlContent, "text/html");
}

private async Task<bool> ShouldShowAppView()
33 changes: 0 additions & 33 deletions test/Altinn.App.Api.Tests/OpenApi/swagger.json
Original file line number Diff line number Diff line change
@@ -2265,39 +2265,6 @@
}
}
},
"/{org}/{app}/set-query-params": {
"get": {
"tags": [
"Home"
],
"summary": "Sets query parameters in frontend session storage",
"parameters": [
{
"name": "org",
"in": "path",
"description": "",
"required": true,
"schema": {
"type": "string"
}
},
{
"name": "app",
"in": "path",
"description": "",
"required": true,
"schema": {
"type": "string"
}
}
],
"responses": {
"200": {
"description": "OK"
}
}
}
},
"/{org}/{app}/instances/{instanceOwnerPartyId}/{instanceGuid}": {
"get": {
"tags": [
Loading