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

Change content from text/plain to text/html if swagger service is a sub-element in SICF #82

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

bilencekic
Copy link

If Swagger service is created as a subelement of a REST service, content type by default comes as text/plain and Swagger.html source codes are printed on browser. UI won't be displayed.

Parent service request handler needs to be skipped to trigger Swagger service( since it is a child element ) as below.

me->if_http_extension~lifetime_rc = if_http_extension~co_lifetime_destroy.
me->if_http_extension~flow_rc = if_http_extension~co_flow_ok_others_mand

Above codes will skip REST handler to be destroyed and child handlers to be triggered. Once Swagger handler is triggered, output can be seen as text/plain for content type. This PR will fix it.

bilencekic and others added 3 commits November 10, 2022 23:38
If Swagger service is created as a subelement of a REST service, content type by default comes as `text/plain` and Swagger.html source codes are printed on browser. UI won't be displayed.
Parent service request handler needs to be skipped to trigger Swagger service( since it is a child element ).
 
` me->if_http_extension~lifetime_rc = if_http_extension~co_lifetime_destroy.`
`  me->if_http_extension~flow_rc = if_http_extension~co_flow_ok_others_mand`
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
fix for abaplint

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant