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

Error prevention for space in URL(adress) #17

Closed
NatTh23 opened this issue Sep 13, 2023 · 2 comments
Closed

Error prevention for space in URL(adress) #17

NatTh23 opened this issue Sep 13, 2023 · 2 comments
Assignees
Labels
follow-up-with/dev Clarification with development needed. invalid This doesn't seem right

Comments

@NatTh23
Copy link

NatTh23 commented Sep 13, 2023

Issue description

Hi
Pls create an error when deploying or even while entering the address in the connection details in case it contains space

Example positive
https:///GWSAMPLE_BASIC/SalesOrderLineItemSet(SalesOrderID='${property.SalesOrderID}',ItemPosition='${property.ItemPosition}')/ToHeader

Example negative
https://6eff9cbetrial-trial.integrationsuitetrial-apim.us10.hana.ondemand.com/6eff9cbetrial/v1/GWSAMPLE_BASIC/SalesOrderLineItemSet(SalesOrderID='${property.SalesOrderID}',ItemPosition='${property.ItemPosition }')/ToHeader

--> The space between
'${property.ItemPosition
&
}')
is deployed and leads to an error.

Feedback Type (Optional)

None

Page Title on SAP Help Portal (prefilled)

HTTP Receiver Adapter

Page URL on SAP Help Portal (prefilled)

https://help.sap.com/docs/PRODUCT_ID/368c481cd6954bdfa5d0435479fd4eaf/2da452effb764b3bb28f8e0a2f5bd480.html?state=PRODUCTION&version=Cloud&locale=de

@Karunaharan Karunaharan self-assigned this Sep 13, 2023
@Karunaharan
Copy link
Contributor

Hi @NatTh23 - thanks for the feedback. I'm looking into the issue.

Regards,
Karuna

@Karunaharan
Copy link
Contributor

Hi @NatTh23,
This is a corner case where camel cases are used in the address. Nevertheless, we are considering your feedback. The concerned engineering teams are informed about having a validation check. Also, please note that "spaces" are allowed in the address field in certain cases.

Closing the issue now. Thanks for taking the time to pass on the useful feedback.

Regards,
Karuna

@SrutiSagaram SrutiSagaram added follow-up-with/dev Clarification with development needed. invalid This doesn't seem right labels Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
follow-up-with/dev Clarification with development needed. invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

3 participants