Attention is currently required from: dexter, jolly. fixeria has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/42833?usp=email )
Change subject: certificates: add checks to make sure cert and key files are readable ...................................................................... Patch Set 1: (3 comments) File src/crypto_utils.erl: https://gerrit.osmocom.org/c/onomondo-eim/+/42833/comment/e859a5c0_20b0b0d9?usp=email : PS1, Line 214: RootCiCertPem = case file:read_file(RootCiCertPath) of cosmetic: you're adding tabs to a file that uses spaces https://gerrit.osmocom.org/c/onomondo-eim/+/42833/comment/95124dca_a1aa1f5a?usp=email : PS1, Line 218: throw `throw` is normally used to break long recursion chains. `erlang:error/1` is a better fit for signalling a run-time error. File src/onomondo_eim_app.erl: https://gerrit.osmocom.org/c/onomondo-eim/+/42833/comment/a64f1537_7b77dcf9?usp=email : PS1, Line 35: true -> likewise: use spaces (not tabs) and `erlang:error/1`. -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42833?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Icaea4aae6188bcdecbc44558aefd2609706be56f Gerrit-Change-Number: 42833 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <[email protected]> Gerrit-Reviewer: jolly <[email protected]> Gerrit-CC: fixeria <[email protected]> Gerrit-Attention: jolly <[email protected]> Gerrit-Attention: dexter <[email protected]> Gerrit-Comment-Date: Tue, 16 Jun 2026 07:31:00 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No
