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

Tokensecurity error on reading properties #1708

Open
pa1pdr opened this issue Mar 31, 2024 · 8 comments
Open

Tokensecurity error on reading properties #1708

pa1pdr opened this issue Mar 31, 2024 · 8 comments

Comments

@pa1pdr
Copy link

pa1pdr commented Mar 31, 2024

On my windows 10 install SignalK v.2.6.2 I get the following error in the log:

TypeError: Cannot read properties of null (reading 'path') at c:\signalk\nodejs\node_modules\signalk-server\lib\tokensecurity.js:468:98 
at Array.find (<anonymous>) at c:\signalk\nodejs\node_modules\signalk-server\lib\tokensecurity.js:467:35 
at Array.find (<anonymous>) at strategy.shouldAllowWrite (c:\signalk\nodejs\node_modules\signalk-server\lib\tokensecurity.js:461:46) 
at processUpdates (c:\signalk\nodejs\node_modules\signalk-server\lib\interfaces\ws.js:366:31)

I've dropped all devices in 'Security', but that doesn't help. Reinstalled signalK: no dice.

@tkurki
Copy link
Member

tkurki commented Mar 31, 2024

@tkurki
Copy link
Member

tkurki commented Mar 31, 2024

Ah. I think something is creating invalid delta values with null in values array. No reconfiguration will fix the issue.

@tkurki
Copy link
Member

tkurki commented Apr 7, 2024

Closing, as this is just an indication of broken input.

@tkurki tkurki closed this as completed Apr 7, 2024
@pa1pdr
Copy link
Author

pa1pdr commented Apr 17, 2024

Ah. I think something is creating invalid delta values with null in values array. No reconfiguration will fix the issue.

I'll scrutinise my SensESP providers. So null is not acceptable as a value and will prevent that from being sent.
Any indicators on the culprits?

@tkurki
Copy link
Member

tkurki commented Apr 19, 2024

Never seen this before.

@tkurki tkurki reopened this Apr 24, 2024
@tkurki
Copy link
Member

tkurki commented Apr 24, 2024

@pa1pdr
Copy link
Author

pa1pdr commented Apr 24, 2024

Sorry: my registration to discord fails.. Do you have any alternate means to communicate?

@tkurki
Copy link
Member

tkurki commented Jun 9, 2024

I just added a pointer to a similar problem, did not mean to redirect this conversation there.

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