juno icon indicating copy to clipboard operation
juno copied to clipboard

Filter user id for authentication

Open peterpeterparker opened this issue 9 months ago • 2 comments

Motivation

Currently, there is no option to filter the list of users by User ID in the Authentication section. This makes it challenging to quickly locate specific users.

What to Do

Add a filter option to the Authentication page that allows users to filter by User ID.

The filter should be placed in a "Filter" popover, following a similar UI/UX pattern as the actions on the Functions and Datastore pages.

Image Image Image

peterpeterparker avatar Feb 27 '25 12:02 peterpeterparker

Can I take the issue?

red-apple38 avatar Feb 27 '25 12:02 red-apple38

Sure, go for it @red-apple38 👍

peterpeterparker avatar Feb 27 '25 15:02 peterpeterparker

would love to pick this up! @peterpeterparker

tinkerer-shubh avatar Jun 05 '25 10:06 tinkerer-shubh

Sure @tinkerer-shubh, go for it 👍

peterpeterparker avatar Jun 05 '25 13:06 peterpeterparker

hey @peterpeterparker,I’ve implemented a filter option for the auth page but it’s still a work in progress. When I try to test it in dev mode, I can’t seem to sign in and get this error:   Canister called ic0.trap with message: <Canister error message

this is what's happening: You have been signed out because your Mission Control could not be certified.

Not sure if I’m missing something—any ideas?

https://internetcomputer.org/docs/references/execution-errors#trapped-explicitly

tinkerer-shubh avatar Jun 05 '25 19:06 tinkerer-shubh

Hi @tinkerer-shubh ,

Did you run the local development environment as described in the HACKING.md ?

  1. You need Docker
  2. In one terminal, run
npm run emulator
  1. In another terminal, run
npm run dev

peterpeterparker avatar Jun 06 '25 04:06 peterpeterparker

yup I did but still the error message stays. there's a longer error that appears which I am not able to look at properly or copy cause it appears and goes too fast(this is in the browser).

also, on the PR checks, the docker checks did not run but I do have docker installed.

tinkerer-shubh avatar Jun 06 '25 05:06 tinkerer-shubh

When you start npm run emulator, is there any issue? i.e. do you see any errors in the log or everything seems fine?

I suspect the default backend to fail at deploying the default modules, which don't happen in my local environment because I keep iterating and building those.

peterpeterparker avatar Jun 06 '25 06:06 peterpeterparker

I think I found a band-aid fix for this, this was the error msg:

Canister called ic0.trap with message: 'juno.cdn.storage.error.missing_releases_description'

I did some digging with my buddy claude and found this as the root cause (could be totally wrong tho):

The console canister validates that assets uploaded to the "releases" collection have a description matching the pattern change=\d+;version=[^;]+$ (e.g., "change=123;version=1.0.0"). During emulator startup, the CLI uploads assets without providing this required description format.

this was the fix and it's working for now :)

Modified src/console/src/cdn/assert.rs to skip description validation in development mode when no description is provided:

// Skip description validation in development mode (when description is None or empty) // This allows the emulator to work without requiring proper release descriptions if description.is_some() && !description.as_ref().unwrap().is_empty() { assert_releases_description(description)?; }

tinkerer-shubh avatar Jun 06 '25 08:06 tinkerer-shubh

Haaaaa you are right. main contains features - and new assertions - that are not compatible with the latest Docker image, which itself will be upgraded, once new features are released.

That's why you needed the workaround, well played 👍.

I understand that you are unblocked but, I'll still try to find a temporary solution.

peterpeterparker avatar Jun 06 '25 09:06 peterpeterparker

Main should be good now. I temporary disabled the new asseration in #1670. Thanks for the ping about it!

peterpeterparker avatar Jun 06 '25 11:06 peterpeterparker