Skip to content

Invalid feedname results in no error client or server side #67

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

Closed
askpatrickw opened this issue Mar 23, 2021 · 4 comments
Closed

Invalid feedname results in no error client or server side #67

askpatrickw opened this issue Mar 23, 2021 · 4 comments
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@askpatrickw
Copy link
Contributor

I was doing this...

_mqtt_key = f"{aio_username}/feeds/{feed_name}"
aio.publish(_mqtt_key, value)

instead of

aio.publish(feed_name, value)

And I got no error client side or in the io.adafruit.com monitoring page.
Took me forever to figure out why it wasn't working... :-)

@brentru
Copy link
Member

brentru commented Mar 24, 2021

@brentru brentru self-assigned this Mar 24, 2021
@brentru
Copy link
Member

brentru commented Mar 26, 2021

@askpatrickw I can confirm this occurs when attempting to publish to io.publish("brubell/feeds/temp", 1) and also with subscribe client.subscribe("brubell/feeds/temperature").

This library should perform validation across any method which access feed keys using the following rules via https://learn.adafruit.com/naming-things-in-adafruit-io/the-two-feed-identifiers:

  1. The only characters we permit are lower case English letters ("a" to "z"), numbers, and dash ("-").
  2. The name must be a maximum length of 128 characters.

@askpatrickw askpatrickw added bug Something isn't working good first issue Good for newcomers labels Mar 26, 2021
@jposada202020 jposada202020 pinned this issue Apr 23, 2021
@jposada202020 jposada202020 unpinned this issue Apr 23, 2021
@jposada202020
Copy link

@askpatrickw I thiink chnges related with this issue we introduced in #68. Let me know if this is the case. Thanks :)

@brentru
Copy link
Member

brentru commented Apr 23, 2021

@jposada202020 Confirming that this issue was address in #68, closing.

@brentru brentru closed this as completed Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants