Adding in Mockoon#88
Conversation
| "responses": [ | ||
| { | ||
| "uuid": "aea874b6-6d35-4fce-8e4b-2ec5d1deaa5d", | ||
| "body": "{\n \"access_token\": \"eyJhbGciOiJub25lIiwidHlwIjoiSldUIn0.eyJpc3MiOiJodHRwOi8vbG9jYWxob3N0OjgwMDAiLCJhdWQiOiJDaGVja1BlcmZvcm1hbmNlRGF0YSIsInN1YiI6Im1vY2tvb24tdXNlci0xMjMiLCJuYW1lIjoiTW9ja29vbiBVc2VyIiwiZ2l2ZW5fbmFtZSI6Ik1vY2tvb24iLCJzdXJuYW1lIjoiVXNlciIsImVtYWlsIjoibW9jay51c2VyQGV4YW1wbGUuY29tIiwiaWF0IjoxNTE2MjM5MDIyLCJleHAiOjE5MTYyMzkwMjIsIm9yZ2FuaXNhdGlvbiI6IntcImlkXCI6XCJtb2Nrb29uLW9yZy00NTZcIixcIm5hbWVcIjpcIk1vY2tvb24gT3JnYW5pc2F0aW9uXCJ9In0.\",\n \"id_token\": \"eyJhbGciOiJub25lIiwidHlwIjoiSldUIn0.eyJpc3MiOiJodHRwOi8vbG9jYWxob3N0OjgwMDAiLCJhdWQiOiJDaGVja1BlcmZvcm1hbmNlRGF0YSIsInN1YiI6Im1vY2tvb24tdXNlci0xMjMiLCJuYW1lIjoiTW9ja29vbiBVc2VyIiwiZ2l2ZW5fbmFtZSI6Ik1vY2tvb24iLCJzdXJuYW1lIjoiVXNlciIsImVtYWlsIjoibW9jay51c2VyQGV4YW1wbGUuY29tIiwiaWF0IjoxNTE2MjM5MDIyLCJleHAiOjE5MTYyMzkwMjIsIm9yZ2FuaXNhdGlvbiI6IntcImlkXCI6XCJtb2Nrb29uLW9yZy00NTZcIixcIm5hbWVcIjpcIk1vY2tvb24gT3JnYW5pc2F0aW9uXCJ9In0.\",\n \"token_type\": \"Bearer\",\n \"expires_in\": 3600\n}", |
There was a problem hiding this comment.
prollly don't want this hard-coded?
|
|
||
| public class OrganisationDto | ||
| { | ||
| public string Id { get; init; } |
There was a problem hiding this comment.
Non-nullable property 'Id' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
There was a problem hiding this comment.
This is resolved in main.
| public string? MetadataAddress { get; init; } | ||
| public string? ClientSecret { get; init; } | ||
| public string BaseUrl { get; set; } = string.Empty; | ||
| public string ApiClientSecret { get; set; } |
There was a problem hiding this comment.
Non-nullable property 'ApiClientSecret' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
There was a problem hiding this comment.
This is resolved in main.
| "Audience": "signin.education.gov.uk", | ||
| "MetadataAddress": "http://localhost:8000/.well-known/openid-configuration", | ||
| "ClientSecret": "eye-am-mockoon-authentication-which-needs-to-be-long", | ||
| "RequireHttpsMetadata": false, |
There was a problem hiding this comment.
having "RequireHttpsMetadata": false means The SignatureValidator accepts any string as a valid token. We probably don't want this as the default. Maybe set it to true, and have local env overrides for it?
| }, | ||
| "DfeSignIn": { | ||
| "BaseUrl": "http://localhost:8000/", | ||
| "ApiClientSecret": "mockoon-api-secret-which-needs-to-be-long", |
There was a problem hiding this comment.
Probably want any Mockoon settings to be in local env config.
Added in config for Mockoon