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

send-message not using default config file for auth #80

Closed
jschollenberger opened this issue Dec 11, 2021 · 2 comments
Closed

send-message not using default config file for auth #80

jschollenberger opened this issue Dec 11, 2021 · 2 comments

Comments

@jschollenberger
Copy link

I'm not sure if I'm using this incorrectly, but I would expect send-message to use the config file by default. The help may imply this is the intended behavior:

-c, --config TEXT               The aprsd config file to use for options.
                                  [default: /root/.config/aprsd/aprsd.yml]
# yq .aprs /root/.config/aprsd/aprsd.yml
{
  "enabled": true,
  "host": "rotate.aprs2.net",
  "login": mycall-15",
  "password": "12345",
  "port": 14580
}

but upon sending a message:

# aprsd send-message mycall-1 hello!
Must set --aprs_login or APRS_LOGIN
# aprsd send-message --config mycall-1 hello!
Usage: aprsd send-message [OPTIONS] TOCALLSIGN COMMAND...
Try 'aprsd send-message -h' for help.

Error: Missing argument 'COMMAND...'.

It does work fine when specifying the aprs login and password on the command line. Is send-message supposed to pull these values from the config? Or do they have to be provided via env variables?

@hemna
Copy link
Collaborator

hemna commented Dec 11, 2021

I can change it so that the fallback is to the use config if not specified.

@jschollenberger
Copy link
Author

That would be great! I'll go ahead and close this since it's not an issue but more of a feature request.

hemna added a commit that referenced this issue Jul 13, 2023
aprslib incorrectly decodes weather packets and doesn't provide
wind_speed or wind_direction from the CSE/SPD 7 bytes in the APRS
packet.  This patch puts a temporary fix in place until the
aprslib pull request lands and is released.

rossengeorgiev/aprs-python#80

rossengeorgiev/aprs-python#81
hemna added a commit that referenced this issue Jul 13, 2023
aprslib incorrectly decodes weather packets and doesn't provide
wind_speed or wind_direction from the CSE/SPD 7 bytes in the APRS
packet.  This patch puts a temporary fix in place until the
aprslib pull request lands and is released.

rossengeorgiev/aprs-python#80

rossengeorgiev/aprs-python#81
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants