Skip to content
This repository has been archived by the owner on Nov 11, 2024. It is now read-only.

dss namespace hidden, secure namespace redacted values #378

Merged
merged 5 commits into from
Dec 2, 2022

Conversation

techcobweb
Copy link
Contributor

@techcobweb techcobweb commented Nov 30, 2022

Signed-off-by: Mike Cobbett 77053+techcobweb@users.noreply.github.com

  • Put unit tests around the CSP servlet
  • Changed some 500 (internal server error) codes to 400 or 404.
  • Added a name validator class to centralise validation of names.
  • Servlet supports hiding of the 'dss' namespace entirely if it's there.
  • Servlet supports redacting all the values when 'security' namespace is queried.
  • Seems to work when tested with curl/soapUI
    • still not working with swagger as a result of CORS stuff I believe.
  • New bug found by inspection
  • Unit tests 80% branch coverage.
  • Added run-locally script for easy invocation of framework components.

Signed-off-by: Mike Cobbett <77053+techcobweb@users.noreply.github.com>
Signed-off-by: Mike Cobbett <77053+techcobweb@users.noreply.github.com>
@galasa-team
Copy link

Build successful

1 similar comment
@galasa-team
Copy link

Build successful

@techcobweb
Copy link
Contributor Author

image
Unit tests now at 80% probably enough for now.

Signed-off-by: Mike Cobbett <77053+techcobweb@users.noreply.github.com>
@galasa-team
Copy link

Build successful

Signed-off-by: Mike Cobbett <77053+techcobweb@users.noreply.github.com>
@galasa-team
Copy link

Build successful

Signed-off-by: Mike Cobbett <77053+techcobweb@users.noreply.github.com>
@galasa-team
Copy link

Build successful

@techcobweb techcobweb merged commit eafa603 into main Dec 2, 2022
@techcobweb techcobweb deleted the techcobweb-security branch December 2, 2022 13:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants