sinricpro / esp8266-esp32-sdk

Library for https://sinric.pro - simple way to connect your device to Alexa, Google Home, SmartThings and cloud
https://sinric.pro
228 stars 121 forks source link

multiple "OK" errors followed by a success #258

Closed DrakeAnglin closed 2 years ago

DrakeAnglin commented 2 years ago

when i send one command to a device, i am getting several red "OK" toast followed by a success.

image image

sivar2311 commented 2 years ago

Which commands do you send? Can you share the code snippet please?

DrakeAnglin commented 2 years ago

this is happening via the portal UI. If i send any command (open, close, on, off, or set %). I immediately receive 3 fails and a success message. you can see in the screenshot the time stamps are all the same.

sivar2311 commented 2 years ago

Can you share the code, pelase?

DrakeAnglin commented 2 years ago

as i mentioned above this is happening on the portal site (portal.sinric.pro/dashboard). This is not my code. My code works fine, i get one request and i handle it. I have 4 devices and they all have the same code and this is only happening on the one device, this started happening as soon as i brought the 4th devices online.

The issue is that I am receiving these error message on your site.

sivar2311 commented 2 years ago

Did you check the used device IDs in your sketches?

DrakeAnglin commented 2 years ago

can you elaborate on that?

I assume you are asking if i have a duplicate ID running on multiple devices. If that is what you're asking then no, i have 4 unique ID's on each device. I am not trying to run the same ID on multiple devices. I did purchase 5 more ID's as the first 3 worked perfectly and i wanted to continue using this service.

sivar2311 commented 2 years ago

I assume you are asking if I have a duplicate ID on multiple devices.

Yes, that was my intention :)

So it doesn't seem to be a bug of the SDK, but maybe a problem with the server or your account. For this reason, I ask you to open a new issue in the non-sdk-issue repository and close it here.

kakopappa commented 2 years ago

@DrakeAnglin Can you check whether function return true at the end?

DrakeAnglin commented 2 years ago

@DrakeAnglin Can you check whether function return true at the end?

yes, i always return true. i basically copied the example and used that with little changes on my end. i would assume that is why the last message is a "success" and after watching the logs on a dev device i only receive one message.

sivar2311 commented 2 years ago

do you make use of delay function or does your code need longer than a few seconds to return from the callback?

DrakeAnglin commented 2 years ago

I do not use any long running scripts or delay when i receive a message. when i receive a message i set a servo position and return true.

sivar2311 commented 2 years ago

Does this also happen with one of the example sketches?

kakopappa commented 2 years ago
  1. Please enable the SDK logs.

To do that add the following line to top of the sketch or uncomment it if’s already in the sketch

define ENABLE_DEBUG

  1. Simulate the problem and copy the log from Arduino Serial Monitor and paste it here.

On Tue, 1 Mar 2022 at 1:24 AM sivar2311 @.***> wrote:

Does this also happen with one of the example sketches?

— Reply to this email directly, view it on GitHub https://github.com/sinricpro/esp8266-esp32-sdk/issues/258#issuecomment-1054539314, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABZAZZSTZM2YJOJQ7IJS3DTU5O4XPANCNFSM5PLATW4A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you commented.Message ID: @.***>

stale[bot] commented 2 years ago

This issue has gone quiet. Spooky quiet. We currently close issues after 14 days of inactivity. It’s been at least 7 days since the last update here. If we missed this issue or if you want to keep it open, please reply here. As a friendly reminder, the best way to fix this or any other problem is to provide a detailed error description including a serial log. Thanks for being a part of the SinricPro community!

stale[bot] commented 2 years ago

Hey again! It’s been 14 days since anything happened on this issue, so our friendly robot (that’s me!) is going to close it. Please keep in mind that I’m only a robot, so if I’ve closed this issue in error, I’m HUMAN_EMOTION_SORRY. Please feel free to comment on this issue or create a new one if you need anything else. As a friendly reminder, the best way to fix this or any other problem is to provide a detailed error description including a serial log. Thanks again for being a part of the SinricPro community!