Skip to content
This repository was archived by the owner on Jun 17, 2024. It is now read-only.

Certificate failure fix#32

Open
4or5trees wants to merge 2 commits intoAzure-Samples:masterfrom
4or5trees:certificate-failure-fix
Open

Certificate failure fix#32
4or5trees wants to merge 2 commits intoAzure-Samples:masterfrom
4or5trees:certificate-failure-fix

Conversation

@4or5trees
Copy link
Copy Markdown

Purpose

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type

What kind of change does this Pull Request introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
    • Prepare Rasberry Pi for tutorial 1/3 (attach relevant modules)
    • Build & Push files for tutorial 1/3
    • Create deployment for single device for tutorial 1/3

Test the code

  • N/A

What to Check

Verify that the following are valid

  • Tutorial 1: Verify that D2C messages can be successfully monitored in VScode
  • Tutorial 3:
    • Verify that D2C messages can be successfully monitored in VScode
    • Verify that commands get recognized (e.g. 'time' or 'joke')

Other Information

Verified that these changes work on my own Raspberry Pi Grove Starter kit.
I also updated the deployment.template.json for tutorial 2 with EdgeHub tag 1.0, even though I did not experience any issue with this tutorial.

@msftclas
Copy link
Copy Markdown

msftclas commented Mar 3, 2019

CLA assistant check
All CLA requirements met.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants