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

Fix for invalid jsonConfig message #397

Closed
wants to merge 1 commit into from

Conversation

mattreim
Copy link
Contributor

@mattreim mattreim commented Oct 3, 2024

warn: admin.0 (822) deconz has an invalid jsonConfig: [{"instancePath":"/items/_instruction/darkStyle","schemaPath":"#/properties/items/patternProperties/%5E.%2B/properties/darkStyle/type","keyword":"type","params":{"type":"string"},"message":"must be string"},{"instancePath":"","schemaPath":"#/if","keyword":"if","params":{"failingKeyword":"else"},"message":"must match \"else\" schema"}]

Nothing has changed in terms of optics.

@mattreim mattreim closed this Oct 3, 2024
@mattreim
Copy link
Contributor Author

mattreim commented Oct 3, 2024

Now I've tried so much that the adapter no longer works.

@mattreim mattreim reopened this Oct 3, 2024
@mattreim
Copy link
Contributor Author

mattreim commented Oct 3, 2024

@Jey-Cee Unfortunately, the adapter no longer works at all when reinstalled. Now I have reinstalled my system and it does not connect to deConz. Downgrading to an older version also doesn't work because the "index_m.html" cannot be found.

@Jey-Cee
Copy link
Owner

Jey-Cee commented Oct 3, 2024

Try an upload of adapter files. Turn on Expert mode, go to Adapter tab, hit the 3 dot button on deConz card and then the arrow up button.

@mattreim
Copy link
Contributor Author

mattreim commented Oct 3, 2024

With an installation of a restore (index_m/io-package) and then an update it works.

@mattreim
Copy link
Contributor Author

mattreim commented Oct 3, 2024

There are no errors in the log because it doesn't connect.

@Jey-Cee
Copy link
Owner

Jey-Cee commented Oct 3, 2024

Which js-controller is running on your host?

@mattreim
Copy link
Contributor Author

mattreim commented Oct 3, 2024

Node.js: v20.17.0
NPM: 10.8.2
js-controller: 6.0.11

@mattreim
Copy link
Contributor Author

mattreim commented Oct 3, 2024

Could it be that “bridge” is no longer connected? Just an idea since port and websocket appear in main.js (with native values).

@Jey-Cee
Copy link
Owner

Jey-Cee commented Oct 3, 2024

Hm on my productive system the adapter works.
On my test system and the productive system is nodejs 22 installed.
I have to test if this is the problem.

@mattreim
Copy link
Contributor Author

mattreim commented Oct 3, 2024

It's clearly a reinstallation problem.

@Jey-Cee
Copy link
Owner

Jey-Cee commented Oct 4, 2024

Check 1.6.2

@mattreim
Copy link
Contributor Author

mattreim commented Oct 4, 2024

It now finds the deConz version and creates the object structure but it still doesn't show me connected.

IOB

It doesn't update either.

@mattreim
Copy link
Contributor Author

mattreim commented Oct 4, 2024

I think it's because of the other "results.native...".

@Jey-Cee
Copy link
Owner

Jey-Cee commented Oct 4, 2024

Can you point to a soecific line in code?

@mattreim
Copy link
Contributor Author

mattreim commented Oct 4, 2024

if (results.native.ipaddress === undefined) {

if (results.native.port === "" || results.native.port === null) {

if (results.native.user === "" || results.native.user === null) {

host = results !== null && results.native.ipaddress !== undefined ? results.native.ipaddress : null;

port = results !== null && results.native.websocketport !== undefined ? results.native.websocketport : 443;

user = results !== null && results.native.user !== undefined ? results.native.user : null;

ip = results.native.ipaddress;

port = results.native.port;

user = results.native.user;

I hope it was all of them.

@Jey-Cee
Copy link
Owner

Jey-Cee commented Oct 4, 2024

Thx. I hope thats all. V1.6.3 is published.

@mattreim
Copy link
Contributor Author

mattreim commented Oct 4, 2024

Now it can be installed again without any problems. Thanks

@mattreim
Copy link
Contributor Author

mattreim commented Oct 4, 2024

Two notes, you should delete the versions that don't work and the invalid jsonConfig error will still exist.

@mattreim
Copy link
Contributor Author

mattreim commented Oct 4, 2024

But otherwise everything is fine.
IOB

@mattreim
Copy link
Contributor Author

mattreim commented Oct 6, 2024

@Jey-Cee I think websocketport should be here.

const port = adapter.config.port ? adapter.config.websocketport : 443;

@Jey-Cee
Copy link
Owner

Jey-Cee commented Oct 6, 2024

We can close this as it is a bug in the schema of darkStyle.
#ioBroker/adapter-react-v5#443

@Jey-Cee Jey-Cee closed this Oct 6, 2024
@mattreim mattreim deleted the Fix-JSON branch October 13, 2024 13:46
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

Successfully merging this pull request may close these issues.

2 participants