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

Add logs for md5 checksum #80

Merged
merged 6 commits into from
Mar 1, 2023
Merged

Conversation

zbud-msft
Copy link
Contributor

@zbud-msft zbud-msft commented Feb 27, 2023

Why I did it

ADO tracking #16605111

Add additional logs for when there is a mismatching public key and private key such as timestamp and md5 checksum

How I did it

Add logs

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@zbud-msft zbud-msft marked this pull request as ready for review February 28, 2023 18:06
@@ -90,6 +91,9 @@ func main() {
}
certificate, err = tls.LoadX509KeyPair(*serverCert, *serverKey)
if err != nil {
currentTime := time.Now()
Copy link
Collaborator

@qiluo-msft qiluo-msft Feb 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now()

always use UTC? #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed to UTC

@qiluo-msft qiluo-msft merged commit 50123ef into sonic-net:master Mar 1, 2023
qiluo-msft pushed a commit that referenced this pull request Mar 1, 2023
#### Why I did it

Add additional logs for when there is a mismatching public key and private key such as timestamp and md5 checksum

#### How I did it

Add logs
@zbud-msft zbud-msft deleted the AddMd5Logs branch March 1, 2023 18:38
qiluo-msft pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Mar 10, 2023
#### Why I did it

update contains following commits

50123ef Zain Budhwani   Tue Feb 28 16:48:22 2023 -0800  Add logs for md5 checksum (sonic-net/sonic-gnmi#80)
a90f2b3 Zain Budhwani   Mon Feb 27 23:44:49 2023 -0800  Add get-update to azp yml (sonic-net/sonic-gnmi#79)
14fe6f4 Zain Budhwani   Tue Jan 31 14:11:27 2023 -0800  Add 202012 branch to pr checker (sonic-net/sonic-gnmi#72)
a792474 Zain Budhwani   Tue Jan 31 09:22:38 2023 -0800  Fix crash when retrieving cpu utilization (sonic-net/sonic-gnmi#70)

#### How I did it

Fetch new changes
xumia pushed a commit to xumia/sonic-buildimage-1 that referenced this pull request Mar 10, 2023
#### Why I did it

update contains following commits

50123ef Zain Budhwani   Tue Feb 28 16:48:22 2023 -0800  Add logs for md5 checksum (sonic-net/sonic-gnmi#80)
a90f2b3 Zain Budhwani   Mon Feb 27 23:44:49 2023 -0800  Add get-update to azp yml (sonic-net/sonic-gnmi#79)
14fe6f4 Zain Budhwani   Tue Jan 31 14:11:27 2023 -0800  Add 202012 branch to pr checker (sonic-net/sonic-gnmi#72)
a792474 Zain Budhwani   Tue Jan 31 09:22:38 2023 -0800  Fix crash when retrieving cpu utilization (sonic-net/sonic-gnmi#70)

#### How I did it

Fetch new changes
qiluo-msft pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Mar 20, 2023
Has following commits:
```
b93c4ac Zain Budhwani Wed Mar 1 15:45:43 2023 -0800 Fix crash when retrieving cpu utilization (sonic-net/sonic-gnmi#70) (sonic-net/sonic-gnmi#71)
af1ec19 Zain Budhwani Wed Mar 1 15:13:53 2023 -0800 Add diff cov (sonic-net/sonic-gnmi#85)
3f41377 Zain Budhwani Tue Feb 28 16:48:22 2023 -0800 Add logs for md5 checksum (sonic-net/sonic-gnmi#80)
67b7fb2 Zain Budhwani Mon Feb 27 23:44:49 2023 -0800 Add get-update to azp yml (sonic-net/sonic-gnmi#79)
5d6c47f Zain Budhwani Fri Feb 24 13:11:53 2023 -0800 Add net core and code coverage results (sonic-net/sonic-gnmi#77)
984bc6d Zain Budhwani Wed Feb 22 16:03:01 2023 -0800 [202012] Enable unit test (sonic-net/sonic-gnmi#76)
e8e4335 Zain Budhwani Fri Feb 10 16:27:58 2023 -0800 Change dir name in pipeline (sonic-net/sonic-gnmi#75)
a1cc7ab Zain Budhwani Tue Jan 31 14:11:27 2023 -0800 Add 202012 branch to pr checker (sonic-net/sonic-gnmi#72)
eaea6c5 ganglyu Mon Nov 14 10:18:07 2022 +0800 Fix format
```
zbud-msft added a commit to zbud-msft/sonic-gnmi that referenced this pull request Jun 8, 2023
#### Why I did it

Add additional logs for when there is a mismatching public key and private key such as timestamp and md5 checksum

#### How I did it

Add logs
yxieca pushed a commit that referenced this pull request Jun 20, 2023
#### Why I did it

Add additional logs for when there is a mismatching public key and private key such as timestamp and md5 checksum

#### How I did it

Add logs
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.

3 participants