kaxil commented on pull request #11468: URL: https://github.com/apache/airflow/pull/11468#issuecomment-707169823
> Should we add --conn-password to this list? > And for bonus points the password part of the URI style? Added `--conn-password`, will address URI style later in a separate PR as we can do some cleanup of other parts where we mask Password in Conn URI too > _technically_ this is a password of `=test` I think, (or it's not actually accepted by argparse) and it would have to be given as `-p test`. > > But I think as an edge case this is okay if we don't worry about it Looks like it works fine though ```python In [1]: from airflow.cli.cli_parser import get_parser In [2]: parser = get_parser() In [8]: parser.parse_args(["users", "create","-e" ,"airflow" , "-p=10", "-r", "Admin", "-f", "as", "-l", "asd", "-u", "as"]) Out[8]: Namespace(email='airflow', firstname='as', func=<function lazy_load_command.<locals>.command at 0x7fd196603dd0>, lastname='asd', password='10', role='Admin', subcommand='create', use_random_password=False, username='as') ``` ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
