-
-
Notifications
You must be signed in to change notification settings - Fork 568
Object Handler of type HTTP #7100
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
Conversation
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Also needs a test, please https://github.com/abapGit-tests/HTTP PS: This closes #5522. Just made it in under 1000 days 😉 |
Test data provided |
It almost works... Error during activation of the HTTP dest: It happens when the system tries to generate the ICF node
mmm, this might take a while to figure out 🤷 |
It looks like |
|
Its a bug in |
Is it just the name of the interface or is it a bigger correction? |
It's a bigger correction. Changing the interface would let you activate the service but not release it and the |
ok. Please test that it works fine with the small changes I made above. We can merge it then. I will add it to our CI when it is upgraded to 758 next year. |
The note 3554855 has been released and should fix the issue for older releases. |
it required a change in the git handler as well to include the created sicf node, see this pull request . |
fixes issue #7099