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

[BUG] Mm5 and Mp1 interfaces of MEC Platform give incorrect location header #77

Closed
gabrik opened this issue May 9, 2019 · 2 comments · Fixed by #78
Closed

[BUG] Mm5 and Mp1 interfaces of MEC Platform give incorrect location header #77

gabrik opened this issue May 9, 2019 · 2 comments · Fixed by #78
Labels
bug Something isn't working

Comments

@gabrik
Copy link
Contributor

gabrik commented May 9, 2019

MEC Platform Mm5 and Mp1 interfaces procuces a wrong URL in the location header upon a the creation of a resource

from a TCP dump


POST /exampleAPI/mm5/v1/applications/123/dns_rules HTTP/1.1
Host: 10.212.26.198:8091
Accept: */*
Connection: keep-alive
Accept-Encoding: gzip, deflate
Content-Length: 125
User-Agent: python-requests/2.9.1

{"dnsRuleId":"dnsRule2","domainName":"googledns.mec","ipAddressType":"IP_V4","ipAddress":"8.8.4.4","ttl":10,"state":"ACTIVE"}HTTP/1.1 201 Created
Server: nginx/1.14.0 (Ubuntu)
Date: Thu, 09 May 2019 10:59:38 GMT
Content-Type: application/json
Transfer-Encoding: chunked
Connection: keep-alive
location: /exampleAPI/mm5/v1/applications123dns_rulesdnsRule2

89
{"DnsRule":{"dnsRuleId":"dnsRule2","domainName":"googledns.mec","ipAddressType":"IP_V4","ipAddress":"8.8.4.4","ttl":10,"state":"ACTIVE"}}
0

Missing / in the URL

@gabrik gabrik added the bug Something isn't working label May 9, 2019
@gabrik
Copy link
Contributor Author

gabrik commented May 9, 2019

Same Bug applies to MEAO Mm1

@gabrik
Copy link
Contributor Author

gabrik commented May 9, 2019

Problem is un functions creating the urls, they are missing /
See: Mp1
Same applies for Mm5 and Mm1

Will fix in next PR with other bug Fixes to MEAO

@gabrik gabrik mentioned this issue May 9, 2019
@gabrik gabrik closed this as completed in #78 May 9, 2019
gabrik added a commit that referenced this issue May 9, 2019
## MEC Fixes
- Fix #77
- Fix bug in Mm5 client adding Headers that allow it to work also if a nginx proxy is used to hidden the MEC Platform Mm5
- Fix MEAO Transport registration during application registration to MEC Platform
gabrik added a commit that referenced this issue Nov 26, 2020
## MEC Fixes
- Fix #77
- Fix bug in Mm5 client adding Headers that allow it to work also if a nginx proxy is used to hidden the MEC Platform Mm5
- Fix MEAO Transport registration during application registration to MEC Platform
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant