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

btpcli issue #172

Closed
gflex opened this issue Aug 11, 2023 · 4 comments
Closed

btpcli issue #172

gflex opened this issue Aug 11, 2023 · 4 comments
Assignees
Labels
contribution Valuable Contribution type/clarity Something was unclear in the documentation. ua-review Under review by the UA team

Comments

@gflex
Copy link

gflex commented Aug 11, 2023

Issue description

btpcli comes with incorrect permissions

Downloaded the file and extracted using tar:

get -q -O btpcli.tar.gz --no-cookies --header "Cookie: eula_3_1_agreed=tools.hana.ondemand.com/developer-license-3_1.txt; path=/;" -S https://tools.hana.ondemand.com/additional/btp-cli-linux-amd64-latest.tar.gz &&
tar xf btpcli.tar.gz && sudo mv linux-amd64/btp /usr/local/bin/ 

then ran ls and saw the huge UID and GID which surpass some linux distros default limits.

root@90c50ca56fb9:/# ls -latr /usr/local/bin/btp
-rwxr-xr-x 1 1672090608 510800513 12190452 Jul 11 09:24 /usr/local/bin/btp

/usr/local/bin/btp --version
SAP BTP command line interface (client v2.49.0)

expected output:

root@90c50ca56fb9:/# ls -latr /usr/local/bin/btp
-rwxr-xr-x 1 root root 12190452 Jul 11 09:24 /usr/local/bin/btp

Please as the colleagues who produce the file to rectify.

Feedback Type (Optional)

None

Page Title on SAP Help Portal (prefilled)

Account Administration Using the SAP BTP Command Line Interface (btp CLI)

Page URL on SAP Help Portal (prefilled)

https://help.sap.com/docs/btp/sap-business-technology-platform/account-administration-using-sap-btp-command-line-interface-btp-cli

@gflex gflex closed this as completed Aug 11, 2023
@gflex gflex reopened this Aug 11, 2023
@NHingerl NHingerl self-assigned this Aug 14, 2023
@NHingerl NHingerl added follow-up-with/dev Clarification with development needed ua-review Under review by the UA team labels Aug 14, 2023
@NHingerl
Copy link
Contributor

Hi @gflex, I learned that you found a solution for the technical problem.

So I'd consider this issue solved, because the request was about a code change, not documentation.
If you desire any change in documentation, please re-open this ticket and specify the required change.

Just to document it here as well, this was the dev response:

The problem is that tar by default preserves UID/GID from the build machines, which can vary depending on the machine used by xMake.
(...)
I see that you have already fixed your issue by reassigning the owner. Alternatively, you could use the additional flag -o while extracting to achieve the same or use a non-root user, see https://linux.die.net/man/1/tar.

@gflex
Copy link
Author

gflex commented Aug 16, 2023

yes, I found a solution to my problem. Thanks for encouraging customers to finding ways for fixing SAP products flaws right from the tar archive.

@NHingerl NHingerl added invalid This doesn't seem right. and removed follow-up-with/dev Clarification with development needed ua-review Under review by the UA team labels Aug 16, 2023
@mgradl
Copy link
Contributor

mgradl commented Aug 16, 2023

Hi @gflex,

thanks for bringing this issue to our attention. I agree that we have to fix this issue in the archive itself by "anonymizing" the tar entries, and we have already started to work on that. Nevertheless we wanted to provide a possible workaround if other users have the same issue, before we can ship the next release or provide a fixed version as hotfix.

So, again, thanks for taking the time to report the issue!

@NHingerl NHingerl added contribution Valuable Contribution and removed invalid This doesn't seem right. labels Aug 16, 2023
@sap-doc-bot
Copy link

sap-doc-bot bot commented Aug 16, 2023

Thank you for your valuable feedback contribution, @gflex! So that we can recognize your contribution in SAP Community, please tell us your SAP Community profile URL in a reply to this comment; don't include any other text, just the URL on its own, like this:

https://people.sap.com/your-user-name

Thanks!

@nikolasimeonov77 nikolasimeonov77 added ua-review Under review by the UA team type/clarity Something was unclear in the documentation. labels Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution Valuable Contribution type/clarity Something was unclear in the documentation. ua-review Under review by the UA team
Projects
None yet
Development

No branches or pull requests

4 participants