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.

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

Fix Azure HTTP functions when you define a contextPath #493

Open
wants to merge 3 commits into
base: 4.1.x
Choose a base branch
from

Conversation

timyates
Copy link
Contributor

@timyates timyates commented Jun 7, 2023

Supersedes #476 with a test

I couldn't get an in-module test to work, so I spun out a test-suite to test it

timyates and others added 3 commits June 7, 2023 12:58
Micronaut Azure HTTP functions do not work if you define a micronaut.server.contextPath.
When defined the context path is duplicated.
@timyates timyates added the type: bug Something isn't working label Jun 7, 2023
@timyates timyates added this to the 4.1.2 milestone Jun 7, 2023
@timyates timyates self-assigned this Jun 7, 2023
@timyates timyates requested a review from sdelamo June 7, 2023 12:29
@timyates timyates linked an issue Jun 7, 2023 that may be closed by this pull request
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

75.0% 75.0% Coverage
0.0% 0.0% Duplication

@timyates
Copy link
Contributor Author

Ping @sdelamo

timyates added a commit that referenced this pull request Jun 21, 2023
@timyates
Copy link
Contributor Author

@sdelamo ping! 🛎️😉

@sdelamo
Copy link
Contributor

sdelamo commented Aug 24, 2023

I think it is better to leave it as it is for Micronaut Framework 3 since this is fixed in Micronaut Framework 4 with the changes to host.json file. Thoughts?

@timyates
Copy link
Contributor Author

Could we merge this in to fix Micronaut 3, but then not merge it up for Micronaut 4?

@CLAassistant
Copy link

CLAassistant commented Feb 7, 2024

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Azure HTTP functions do not work if you define a contextPath
4 participants