[PR] Show the certificate subject in the certificate list

["arpitjain099 (via GitHub)" <gi...@apache.org>]

arpitjain099 opened a new pull request, #4842:
URL: https://github.com/apache/streampipes/pull/4842

   Closes #4841.
   
   `certificate-label.component.html` rendered `certificate.issuerDn`, so every certificate signed by the same intermediate or root CA showed the same string and there was no way to tell two servers apart without opening each one.
   
   It now shows `subjectDn`, and the OPC UA ApplicationUri underneath it, taken from the `URI` entry of `subjectAlternativeNames` as you suggested, since that is what an operator actually matches a server on. The DNS badges below are untouched. `subjectDn` and `subjectAlternativeNames` are both already on the `Certificate` model (`streampipes-model.ts:721-738`), so nothing needed to change on the API side.
   
   One thing outside the strict scope: `ngOnInit` had a leftover `console.log(this.dnsNames)` sitting in the lines I was editing, so it is gone too. Happy to put it back if you would rather keep this to the reported change.
   
   On verification, so the claim is not larger than what I did: this is a template and component change and I have not run the UI. I checked the field names against the generated model and ran the repo's own prettier config (3.7.4, `ui/.prettierrc.json`) over both files, which reports "All matched files use Prettier code style". I have not run `ng lint` or built the frontend, so CI is the real check there.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@streampipes.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Show the certificate subject in the certificate list

["dominikriemer (via GitHub)" <gi...@apache.org>]

dominikriemer commented on PR #4842:
URL: https://github.com/apache/streampipes/pull/4842#issuecomment-5445326237

   Thank you @arpitjain099 !


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@streampipes.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Show the certificate subject in the certificate list

["dominikriemer (via GitHub)" <gi...@apache.org>]

dominikriemer merged PR #4842:
URL: https://github.com/apache/streampipes/pull/4842


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@streampipes.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org