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.

Sign up for GitHub

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

Jump to bottom

fix: zap security testing #692

Merged
merged 20 commits into from
May 2, 2024
Merged

fix: zap security testing #692

merged 20 commits into from
May 2, 2024

Conversation

GHkrishna
Copy link
Contributor

@GHkrishna GHkrishna commented Apr 30, 2024 •
edited
Loading

What

  • Resolved issues encountered during zap testing

Why

  • To improve validation

KulkarniShashank and others added 18 commits April 15, 2024 15:21
fix: added the validation for the pageNumber and pageSize
bd3cef5 
Signed-off-by: KulkarniShashank <shashank.kulkarni@ayanworks.com>
fix: added the organization name validation
264074e 
Signed-off-by: KulkarniShashank <shashank.kulkarni@ayanworks.com>
@GHkrishna
fix: validation for UUID fixed for connectionId in proof
52ce4ab 
Signed-off-by: Krishna <krishna.waske@ayanworks.com>
@GHkrishna
Merge branch 'fix/zap-security' of https://github.com/credebl/platform …
8c21cf8 
…into fix/zap-security
@GHkrishna
fix: path traversal for updating org details
84864fc 
Signed-off-by: Krishna <krishna.waske@ayanworks.com>
fix: added the validation in connection and ecosystem module for sql-…
d3b9519 
…injection

Signed-off-by: KulkarniShashank <shashank.kulkarni@ayanworks.com>
Merge branch 'fix/zap-security' of https://github.com/credebl/platform …
8ce1d2b 
…into fix/zap-security
Merge branch 'develop' of https://github.com/credebl/platform into fi…
f431cba 
…x/zap-security
@GHkrishna
Merge branch 'fix/zap-security' of https://github.com/credebl/platform …
a14ca7b 
…into fix/zap-security
fix: added the decode URI component in main.ts
d7e9cfa 
Signed-off-by: KulkarniShashank <shashank.kulkarni@ayanworks.com>
Merge branch 'fix/zap-security' of https://github.com/credebl/platform …
15c649c 
…into fix/zap-security
fix: added the messages for the invalid URL
d1365c4 
Signed-off-by: KulkarniShashank <shashank.kulkarni@ayanworks.com>
@GHkrishna
fix: logo uri validation in image update for org for security testing
b7236cd 
Signed-off-by: Krishna <krishna.waske@ayanworks.com>
@GHkrishna
Merge branch 'fix/zap-security' of https://github.com/credebl/platform …
7a0d9d0 
…into fix/zap-security
@GHkrishna
fix: update validation for empty string for logo
1b413ef 
Signed-off-by: Krishna <krishna.waske@ayanworks.com>
@GHkrishna
fix: name validation in update org
07237f4 
Signed-off-by: Krishna <krishna.waske@ayanworks.com>
@GHkrishna
Merge branch 'develop' of https://github.com/credebl/platform into fi…
159098c 
…x/zap-security
@GHkrishna
add: zap changes merged from main
f86d35f 
Signed-off-by: Krishna <krishna.waske@ayanworks.com>
@GHkrishna GHkrishna requested review from shitrerohit and KulkarniShashank and removed request for KulkarniShashank April 30, 2024 11:12
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented May 2, 2024

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@GHkrishna GHkrishna merged commit 2853afa into develop May 2, 2024
2 checks passed
@GHkrishna GHkrishna deleted the fix/zap-security branch June 4, 2024 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shitrerohit shitrerohit Awaiting requested review from shitrerohit

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@GHkrishna