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.

Sign up for GitHub

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

Jump to bottom

Bug: Sending command to a sensor #362

Closed
adcoly opened this issue Mar 1, 2022 · 2 comments · Fixed by #378
Closed

Bug: Sending command to a sensor #362

adcoly opened this issue Mar 1, 2022 · 2 comments · Fixed by #378
Assignees
@kbeaugrand
Labels
bug Something isn't working
Milestone
S13

Comments

@adcoly
Copy link

adcoly commented Mar 1, 2022

Expected Behavior

The command should be received by the sensor.

Current Behavior

In the application insight, when the portal is trying to send the command to the function app, a '403' error happend (screen capture below).

Steps to Reproduce

Sending a command to a real sensor through the portal.

Context (Environment)

Portal version:
1.0.3

Logs

send_command_app_insight_error_403
@adcoly adcoly added the bug Something isn't working label Mar 1, 2022
@adcoly
Copy link
Author

adcoly commented Mar 1, 2022

After a conversation with @kbeaugrand it seems it is a configuration problem coming from our portal or function app (it works well on his simulation environment).

@kbeaugrand
Copy link
Member

The issue is due to HTTP 403 between the portal avec to Function app.

After looking to the configuration it seems that the function app code configured to the portal is good.
To my opinion, the problem might be due to firewall authentication (the outbound IP address for the portal might not be the private endpoint IP)...

We have to check that

@kbeaugrand kbeaugrand added this to the S13 milestone Mar 3, 2022
@kbeaugrand kbeaugrand self-assigned this Mar 3, 2022
@kbeaugrand kbeaugrand moved this to :construction: In Progress in IoT Hub Portal Mar 3, 2022
kbeaugrand added a commit that referenced this issue Mar 3, 2022
@kbeaugrand
Fix #362 - Update API version + Better treat errors
1a0ebc1
@kbeaugrand kbeaugrand linked a pull request Mar 3, 2022 that will close this issue
Fix #362 - Update API version + Better treat errors #378
Merged
9 tasks
audserraCGI added a commit that referenced this issue Mar 4, 2022
@audserraCGI @kbeaugrand
Set device tags on device create/details screens (#366)
b1e1f0f 
* Fix Namespace issue

* Added DeviceTagService + Custom tags on CreateDevicePage

* Customised tag on DeviceDetailPage

* List<DeviceTag> changed to List<string> in CreateDeviceDetails()

* Fix DeviceMapper Tests to work with new DeviceDetails model

* Fix DevicesControllerTests with new DeviceDetails model

* Added DeviceTagService Test

* Added DEviceTagsSettingsController test

* Fix Namespace issue

* Added DeviceTagService + Custom tags on CreateDevicePage

* Customised tag on DeviceDetailPage

* List<DeviceTag> changed to List<string> in CreateDeviceDetails()

* Fix DeviceMapper Tests to work with new DeviceDetails model

* Fix DevicesControllerTests with new DeviceDetails model

* Added DeviceTagService Test

* Added DEviceTagsSettingsController test

* Fixed typo when merging

* Fix code scanning errors

* Small fix

* Added form validation on front side

* Added form validation on Edit page

* Fix comments

* Fix #362 - Update API version + Better treat errors (#378)

* Fix #308 - Remove useless pages in table queries (#379)

* Fix #308 - Remove useless pages in table queries

* Add some unit test to increase code coverage

* Fix tests and clear code

* Fix change in pagination conflict

* Fix Namespace issue

* Added DeviceTagService + Custom tags on CreateDevicePage

* Customised tag on DeviceDetailPage

* List<DeviceTag> changed to List<string> in CreateDeviceDetails()

* Fix DeviceMapper Tests to work with new DeviceDetails model

* Fix DevicesControllerTests with new DeviceDetails model

* Added DeviceTagService Test

* Added DEviceTagsSettingsController test

* Added DeviceTagService + Custom tags on CreateDevicePage

* List<DeviceTag> changed to List<string> in CreateDeviceDetails()

* Added DeviceTagService Test

* Added DEviceTagsSettingsController test

* Fixed typo when merging

* Fix code scanning errors

* Small fix

* Added form validation on front side

* Added form validation on Edit page

* Fix comments

* Fix tests and clear code

* Fix change in pagination conflict

* Update src/AzureIoTHub.Portal/Server/Controllers/v1.0/DeviceTagSettingsController.cs

Co-authored-by: Kevin BEAUGRAND <[email protected]>

Co-authored-by: Kevin BEAUGRAND <[email protected]>
@github-actions github-actions bot mentioned this issue Mar 4, 2022
Merged
@audserraCGI audserraCGI moved this from :construction: In Progress to :rocket: Ready in IoT Hub Portal Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kbeaugrand kbeaugrand

Labels
bug Something isn't working
Projects
IoT Hub Portal
Status: 🚀 Ready
Milestone
S13
Development

Successfully merging a pull request may close this issue.

Fix #362 - Update API version + Better treat errors CGI-FR/IoT-Hub-Portal
2 participants
@kbeaugrand @adcoly