Sprint 22 #131
Merged
Sprint 22 #131
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… Authorization header -Regenerate mocks -Registered Authorization Middleware -Added test for authorization middleware -Added some functionality to Authorization -Made a new yaml file for security testing
- E2Etests Working on enum
… Authorization header -Moved authorization middleware to standard controller file as well as test to that test file
added get api bdd test added bare function for api discovery made fixes to feature file formatting
* Moved the client lib setup to outside the middleware func * Updated test to use test keys for an unused account (develoeprs needs to have an environment variable OAUTH_TEST_KEY. This is set in the pipeline)
added ability to set expected content type in the context so it can be checked in the api discovery handler
added test for response type in context
added attach discovery handler test
- Working on enum
feature/Weos 1247 - API Discovery
… Authorization header -Added the correct openidconnecturl in the test yaml file -Added X-USER-ID contextkey -Added a securitycheck flag -Added a for loop for getting the security schemes
…ware on operation -Added globalinitializers interface -Added for loop to run through global initializers before other initializers -Add function to register and get global intializers -Added global middleware initializer and test
…ware on operation -Fix Module test
…ware on operation -Added an extension for skipexpirycheck -Added code to set the skipexpirycheck -Added validation check for the openidconnecturl
…ware on operation & Create a global initializer for setting the middleware on all operations -Added check for security in order to know when to add the authorization middleware
made default endpoints for returning swagger ui as json and html
feature: WEOS-1343 Create middleware for handling JWT in the incoming Authorization header
…ware on all operations -Take auth token from the request header since the authorization middleware runs first -Made sure the operation middleware uses global context
- Updated yaml - Updated isValid - Added isEnumValid - Added tests
… Connect security scheme -Update the check for errors
# Conflicts: # controllers/rest/openapi_extensions.go # controllers/rest/weos_mocks_test.go # model/mocks_test.go
… Connect security scheme -Debug errors
… Connect security scheme -Debug errors
… Connect security scheme -Debug errors
- Adjusted yaml files - Editted tests and IsEnumValid
- Fixes
- Commented failing test temporarily
- Uncommented failing tests
- Fixed failing tests
Feature/weos 1116 - As a developer I should be able to add an enumeration field to a content type
- Updated enumeration declaration
Updated null declaration in enumeration
…be returned if there is no controller set -Updated the functionality and test so that no error is returned when an incorrect accept header or no header is passed
…be returned if there is no controller set -Debug end to end test and ensure passing
…be returned if there is no controller set -Debug end to end test and ensure passing -Updated feature file -Updated functionality to return a html page -Added custom json marshaller
# Conflicts: # end2end_test.go
…be returned if there is no controller set -Updated test fixture file
…be returned if there is no controller set -Updated JSONMarshall to remove special characters
feature: WEOS-1343 As a developer I should be able to setup an OpenID Connect security scheme
# Conflicts: # context/context.go # controllers/rest/api.go # controllers/rest/controller_standard_test.go # controllers/rest/utils.go # end2end_test.go
feature: WEOS-1365 As a developer I should be able set an example to be returned if there is no controller set
RandyDeo
approved these changes
Mar 4, 2022
shaniah868
approved these changes
Mar 4, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Features
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No