Documentation for Publish API needs extra info #1678

Closed
opened 2018-06-23 03:29:14 +02:00 by 42degrees · 2 comments
42degrees commented 2018-06-23 03:29:14 +02:00 (Migrated from github.com)

Looking at the documentation at: https://lbryio.github.io/lbry/ for "publish" API (and CLI, presumably) the 'name' field apparently has limitations on what characters can be in the field, but these are not documented. I have discovered through trial and error that at least spaces and commas are not allowed. I don't know yet if other characters are also disallowed.

Looking at the documentation at: https://lbryio.github.io/lbry/ for "publish" API (and CLI, presumably) the 'name' field apparently has limitations on what characters can be in the field, but these are not documented. I have discovered through trial and error that at least spaces and commas are not allowed. I don't know yet if other characters are also disallowed.
42degrees commented 2018-06-23 04:01:13 +02:00 (Migrated from github.com)

After digging through the code in lbry/Daemon.py and lbryschema/uri.py I finally found the regular expression governing names is:
valid_name_char = "[a-zA-Z0-9-]"

That is pretty limiting and really should be documented.

After digging through the code in lbry/Daemon.py and lbryschema/uri.py I finally found the regular expression governing names is: valid_name_char = "[a-zA-Z0-9\-]" That is pretty limiting and really should be documented.
tzarebczan commented 2018-06-23 07:04:44 +02:00 (Migrated from github.com)

Issue moved to lbryio/lbry #1268 via ZenHub

Issue moved to [lbryio/lbry #1268](https://github.com/lbryio/lbry/issues/1268) via [**ZenHub**](https://www.zenhub.com/)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#1678
No description provided.