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

Inaccurate Error Message for Already Activated atSign #576

Closed
TylerTrott opened this issue Nov 17, 2023 · 4 comments
Closed

Inaccurate Error Message for Already Activated atSign #576

TylerTrott opened this issue Nov 17, 2023 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@TylerTrott
Copy link
Contributor

TylerTrott commented Nov 17, 2023

Describe the bug

Currently, the software displays an error message indicating activation failure due to an "Invalid data provided by user" and "... is an invalid atSign" when attempting to activate an atSign that is already registered and activated. This error message can be misleading in the scenario where a user is not sure if their atSign is already activated or not.

When I use cmd echo $?, 0 is outputted.

Steps to reproduce

On client terminal:

  1. bash -c "$(curl -fsSL https://getsshnp.noports.com)"
  2. Entered client address (already activated address)
  3. Entered device address (also already activated)
  4. Entered 'am'
  5. export PATH="$HOME/.local/bin:$PATH";
  6. at_activate -a @client_address_key (already activated key)

Expected behavior

The software should recognize when an atSign is already activated and provide a specific error message indicating that the atSign is already registered, rather than displaying a generic "Invalid data provided by user" message.

Screenshots

Screenshot 2023-11-17 at 10 35 47 AM
@TylerTrott TylerTrott added the bug Something isn't working label Nov 17, 2023
@XavierChanth
Copy link
Member

If an atSign is already activated, we currently rely on this program returning a zero exit-code so that we know that the atSign is activated after the run. Useful for docker orchestration with an activate container.

@srieteja
Copy link

@TylerTrott from the screenshot in the issue description, I see one issue. It's possible that '@wildgreen_client_key' is not a valid atsign. at_activate only requires the atsign. Alternatively, you can provide your cram key with '-c' if you have access to one.

@XavierChanth
Copy link
Member

XavierChanth commented Dec 16, 2023

@TylerTrott from the screenshot in the issue description, I see one issue. It's possible that '@wildgreen_client_key' is not a valid atsign. at_activate only requires the atsign. Alternatively, you can provide your cram key with '-c' if you have access to one.

Good catch @srieteja, I think the atSign is @wildgreen_client

@srieteja
Copy link

I just checked with the root server and you are right @XavierChanth

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants