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

[Bug]: Bad error message in GUI when unauthorized and trying to display README.md #7756

Closed
arielshaqed opened this issue May 13, 2024 · 0 comments · Fixed by #8004
Closed
Labels
area/UI Improvements or additions to UI bug Something isn't working good first issue Good for newcomers

Comments

@arielshaqed
Copy link
Contributor

What happened?

What actually happened, including error codes if applies.

Steps to Reproduce:

  1. Configure (on lakeFS.Cloud) your user not to be allowed to read README.md. Stanza for policy:
    {
      "action": [
        "fs:*"
      ],
      "effect": "deny",
      "resource": "arn:lakefs:fs:::repository/*/README.md"
    }
  2. lakectl gives correct error message:
    $ lakectl fs cat lakefs://sample-repo/main/README.md
    insufficient permissions
    401 Unauthorized
  3. WebUI gives minified or otherwise weird error message:
    "objects" tab trying to show README.md but actually showing a pink error box saying "Kn"

Expected behavior

Pink error box saying "Insufficnet permissions" or similar.

lakeFS version

1.22.0

How lakeFS is installed

lakeFS cloud dev installation

Affected clients

webui

Relevant log output

No response

Contact details

[email protected]

@arielshaqed arielshaqed added bug Something isn't working contributor area/UI Improvements or additions to UI and removed contributor labels May 13, 2024
@eladlachmi eladlachmi added the good first issue Good for newcomers label May 28, 2024
@ItamarYuran ItamarYuran linked a pull request Jul 24, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/UI Improvements or additions to UI bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants