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

added more logging to certificate import #6085

Merged
merged 3 commits into from
Feb 8, 2022

Conversation

vipeller
Copy link
Contributor

@vipeller vipeller commented Feb 4, 2022

In order to spot easier if any regression happened with removing bouncy castle, added some more logging. Also MainAsync() is not reports if it stops by exception.

varunpuranik
varunpuranik previously approved these changes Feb 6, 2022
@kodiakhq
Copy link
Contributor

kodiakhq bot commented Feb 7, 2022

This PR currently has a merge conflict. Please resolve this and then re-add the ready-to-merge label.

@kodiakhq kodiakhq bot removed the ready-to-merge label Feb 7, 2022
…ort_log

# Conflicts:
#	edge-hub/core/src/Microsoft.Azure.Devices.Edge.Hub.Service/Program.cs
@kodiakhq kodiakhq bot merged commit 49d41df into Azure:master Feb 8, 2022
vipeller added a commit to vipeller/iotedge that referenced this pull request Feb 8, 2022
vipeller added a commit to vipeller/iotedge that referenced this pull request Feb 8, 2022
damonbarry added a commit to damonbarry/iotedge that referenced this pull request Apr 15, 2022
In order to spot easier if any regression happened with removing bouncy castle, added some more logging. Also MainAsync() is not reports if it stops by exception.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants