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: [CF-924] jetty list webapp dirs #651

Merged
merged 1 commit into from
Jan 7, 2025
Merged

fix: [CF-924] jetty list webapp dirs #651

merged 1 commit into from
Jan 7, 2025

Conversation

frisonisland
Copy link
Contributor

  • Avoid listing files and directories under webapp
  • directly point to web.xml as webapp parameter

@frisonisland
fix: point to web.xml wqhen specifying webappcontext webapp parameter
b476bab 
- deny listing folders inside webapp location
@frisonisland frisonisland marked this pull request as ready for review January 7, 2025 14:33
@frisonisland frisonisland requested a review from a team as a code owner January 7, 2025 14:33
@sonarqube-zextras Sonarqube Zextras
Copy link

Passed Quality Gate passed

Issues

Measures

Project ID: zextras:zm-mailbox

View in SonarQube

@frisonisland frisonisland merged commit 10e40f9 into main Jan 7, 2025
4 checks passed
@frisonisland frisonisland deleted the hotfix/CF-924-jetty-list-webapp-dirs branch January 7, 2025 14:44
frisonisland added a commit that referenced this pull request Jan 9, 2025
@frisonisland
fix: [CF-924] point to web.xml wqhen specifying webappcontext webapp …
3610a2f 
…parameter (#651)

- deny listing folders inside webapp location

(cherry picked from commit 10e40f9)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@keshavbhatt keshavbhatt keshavbhatt approved these changes

@zextrasGiovanniDefacci zextrasGiovanniDefacci zextrasGiovanniDefacci approved these changes

@aheeva-yuliya aheeva-yuliya aheeva-yuliya approved these changes

@sonersivri sonersivri Awaiting requested review from sonersivri sonersivri is a code owner automatically assigned from zextras/kc

@giuliano176 giuliano176 Awaiting requested review from giuliano176

@FrancescoGottardi FrancescoGottardi Awaiting requested review from FrancescoGottardi

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

Successfully merging this pull request may close these issues.
4 participants
@frisonisland @keshavbhatt @zextrasGiovanniDefacci @aheeva-yuliya