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

E-Mail addresses with dots #17

Closed
reloxx13 opened this issue Nov 26, 2019 · 10 comments
Closed

E-Mail addresses with dots #17

reloxx13 opened this issue Nov 26, 2019 · 10 comments
Assignees
Labels
bug Something isn't working

Comments

@reloxx13
Copy link

reloxx13 commented Nov 26, 2019

Heya,

email addresses with (dots) in it are confusing the object lists in iobroker.

e.g. max.mustermann@muster.org

will end in a structure like
plex.0 -> users -> max -> mustermann@muster -> org ->userid

i suggest to just use the userid as object or replace all dots with underscores by creating the objects.

Pi4b debian buster
Node.js: v10.17.0
NPM: 6.12.0
Adapter Version: 0.8.4
JS-Controller: 2.1.1
Admin 3.7.1

@reloxx13 reloxx13 changed the title E-Mail addresse with special chars E-Mail addresses with dots Nov 26, 2019
@Zefau
Copy link
Collaborator

Zefau commented Dec 1, 2019

Thanks for reporting. Could you try v0.8.5 if that fixes the issue?

@Zefau
Copy link
Collaborator

Zefau commented Dec 4, 2019

@reloxx13

@reloxx13
Copy link
Author

reloxx13 commented Dec 4, 2019

Heya,

nope, dots are still breaking the tree.

plex.0 -> users -> firstname -> lastname@test -> com => no data
real email firstname.lastname@test.com

deleted all plex.0 tree and stopped/started plex instance

@Zefau
Copy link
Collaborator

Zefau commented Dec 4, 2019

Are you running the latest version v0.8.7 ? The email has been replaced by the username now actually, so I'm wondering why this is not applied for you.

https://github.com/Zefau/ioBroker.plex/blob/bc2ab1c1636faf33dd5dd47995ca5f9aba15e4d9/plex.js#L922

@reloxx13
Copy link
Author

reloxx13 commented Dec 4, 2019

yes.

some users dont have a username. username is not a required field by plex registering form.

grafik

@Zefau
Copy link
Collaborator

Zefau commented Dec 5, 2019

Ok, thanks for the info. Please try v0.8.8 as this removes the dots now.

@reloxx13
Copy link
Author

reloxx13 commented Dec 6, 2019

:(

grafik

only one user without email now

@Zefau
Copy link
Collaborator

Zefau commented Dec 14, 2019

Could you try v0.8.9? Thanks

@Zefau Zefau self-assigned this Dec 16, 2019
@Zefau Zefau added the bug Something isn't working label Dec 16, 2019
@Zefau
Copy link
Collaborator

Zefau commented Dec 20, 2019

@reloxx13

@reloxx13
Copy link
Author

Heya,

thank you, it looks fine now!

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

2 participants