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

fix(jans-cli-tui): asset upload showing path error #10074

Closed
manojs1978 opened this issue Nov 7, 2024 · 4 comments · Fixed by #10107
Closed

fix(jans-cli-tui): asset upload showing path error #10074

manojs1978 opened this issue Nov 7, 2024 · 4 comments · Fixed by #10107
Assignees
Labels
comp-jans-cli-tui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality

Comments

@manojs1978
Copy link
Contributor

Describe the bug
asset upload showing path error

To Reproduce
Steps to reproduce the behavior:

  1. install jans 1.1.6.nightly
  2. upload properties file in jans-auth asset
  3. edit uploaded asset and change jans-service to config-api
  4. See error

Expected behavior
should not get 500 error,
if existing services is not configured then should get proper error message

Screenshots
If applicable, add screenshots to help explain your problem.
Image

Desktop (please complete the following information):

  • OS: ubuntu
  • Browser [e.g. c
  • Version [e.g. 22]
    jans 1.1.6.nightly
    DB -pgsql
    Smartphone (please complete the following information):
  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@mo-auto mo-auto added comp-jans-cli-tui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality labels Nov 7, 2024
@devrimyatar
Copy link
Contributor

@pujavs This is config-api issue, when TUI puts asset data to config-api, it re-sends all data. See line 10, there is double slash (//)

Image

@pujavs
Copy link
Contributor

pujavs commented Nov 11, 2024

@manojs1978 request you to please share exact details that were send to config-api from TUI. Did you provide any path ?

@manojs1978
Copy link
Contributor Author

@devrimyatar
Copy link
Contributor

@pujavs It is definetly config-api issue. See how config-api stores jansFilePath in database (it begins double slash):

Image

TUI does not send filePath when it post asset, but does when it puts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-jans-cli-tui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants