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

[LLRP] Discovery command on RRFID gateway gives error with status code 202 #263

Open
vyshali-chitikeshi opened this issue May 31, 2023 · 1 comment
Labels
1-low priority denoting isolated changes bug Something isn't working
Milestone

Comments

@vyshali-chitikeshi
Copy link

🐞 Bug Report

Affected Services [REQUIRED]

The issue is located in: RFID device service

Is this a regression?

Not sure

Description and Minimal Reproduction [REQUIRED]

Discovery command on RFID gateway UI gives below error:
'Request failed. Status code of 202'

Repro steps:

  1. Setup RFID gateway hardware as per documentation

  2. Deploy the RFID device and app-inventory service with latest V3 code.

  3. Go to http://127.0.0.1:59711/ RFID gateway UI
    a) Right click on webpage, click on 'inspect' to open browser console
    b) If CORS is not enabled on consul (by default CORS is disabled), notice that it gives CORS related errors on console but no
    errors on UI.
    c) Enable CORS on consul and set corsallowed origin on consul. Find attached consul screesnhtos.
    d) Restart rfid-device service
    e) Refresh RFID gateway UI
    f) Open browser console
    g) Click on 'Discovery'

    Actual behavior: It gives request failed error with status code of 202 on UI and also on browser console.
    Expected behavior: The UI should not complain about the 202 status code
    In the debug logs it shows 'Discover triggered' message correctly,

    Find attached debug logs, UI screenshots before and after enabling CORS.
    UI_before_enabling_CORS'
    UI_after_enabling_CORS
    CORS_allowed_origin
    Screenshot from 2023-05-30 16-35-17
    _edgex-device-rfid-llrp_logs (2).txt

🔥 Exception or Error





🌍 Your Environment

Deployment Environment:

EdgeX Version [REQUIRED]:

Anything else relevant?

@vyshali-chitikeshi vyshali-chitikeshi added the bug Something isn't working label May 31, 2023
@lenny-goodell lenny-goodell added this to the Napa milestone May 31, 2023
@lenny-goodell lenny-goodell added the 1-low priority denoting isolated changes label May 31, 2023
@lenny-goodell lenny-goodell changed the title Discovery command on RRFID gateway gives error with status code 202 [LLRP] Discovery command on RRFID gateway gives error with status code 202 May 31, 2023
@ajcasagrande
Copy link
Contributor

HTTP 202 means Accepted and is a success response. Likely we just are checking for 200 OK only, and instead should check for 2xx or 200 and 202.

@lenny-goodell lenny-goodell transferred this issue from edgexfoundry/device-rfid-llrp-go Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-low priority denoting isolated changes bug Something isn't working
Projects
Status: Icebox
Development

No branches or pull requests

3 participants