feat: Implement working draft of importing existing certificates #15

2024-11-22T19:26:06Z

MarkL4YG commented

2024-11-22 19:26:06 +00:00

(Migrated from github.com)

We are now able to import existing x509v3 certificates into our repository
While doing this PR, significant refactoring requirements arose, because CertificateRequest is actually not a good abstraction. I will refactor this later.

- We are now able to import existing x509v3 certificates into our repository - While doing this PR, significant refactoring requirements arose, because ``CertificateRequest`` is actually not a good abstraction. I will refactor this later.

MarkL4YG commented

2024-11-22 20:06:18 +00:00

(Migrated from github.com)

The relevant function was broken in openssl 3.0.3 and lower since the function set_dateopt (which you quote) always returned 0:

AAHHH, fml. In der Pipeline läuft 3.0.2 😠

> The relevant function was broken in openssl 3.0.3 and lower since the function set_dateopt (which you quote) always returned 0: AAHHH, fml. In der Pipeline läuft 3.0.2 😠

😕 1

torge-hmn (Migrated from github.com) approved these changes 2024-11-22 22:16:59 +00:00

torge-hmn (Migrated from github.com) reviewed 2024-11-22 23:48:09 +00:00

src/test/java/de/mlessmann/certassist/repositories/CertificateRepositoryTest.java

					
				@ -20,9 +22,15 @@ class CertificateRepositoryTest {

				    @Autowired

				    private CertificateExtensionRepository extensionRepository;

				    @BeforeEach

torge-hmn (Migrated from github.com) commented

2024-11-22 23:48:09 +00:00

Die Bulk-Operation ist kein Problem. Es scheint ein Problem beim Column-Mapping zu geben. Ich habe schon ein paar Sachen recherchiert und ausprobiert, aber bekomme es nicht gefixt. Interessanter Weise ist der Test in der IDE-Ausführung immer grün, auch mit dem Test-Profil. Es bricht nur der Gradle-Test ab, was ich auch lokal reproduzieren kann. Interessanter Weise kommt es gelegentlich zu einer erfolgreichen Test-Ausführung, sodass vermutlich irgendwo noch ein Reihenfolge oder Caching-Problem vorliegt.

MarkL4YG (Migrated from github.com) reviewed 2024-11-23 09:48:21 +00:00

src/test/java/de/mlessmann/certassist/repositories/CertificateRepositoryTest.java

					
				@ -20,9 +22,15 @@ class CertificateRepositoryTest {

				    @Autowired

				    private CertificateExtensionRepository extensionRepository;

				    @BeforeEach

MarkL4YG (Migrated from github.com) commented

2024-11-23 09:48:21 +00:00

Interessant. Ich werde mir das noch weiter anschauen. Die Bulk-Operation war halt in der Zeile, die in einer der Exceptions aufgetaucht ist. Aber dann war das wohl etwas unterliegendes.
Jedenfalls zeigt sich einmal mehr, dass das Logging noch unzureichend ist. 😅

Interessant. Ich werde mir das noch weiter anschauen. Die Bulk-Operation war halt in der Zeile, die in einer der Exceptions aufgetaucht ist. Aber dann war das wohl etwas unterliegendes. Jedenfalls zeigt sich einmal mehr, dass das Logging noch unzureichend ist. 😅

MarkL4YG (Migrated from github.com) reviewed 2024-11-23 10:42:25 +00:00

src/test/java/de/mlessmann/certassist/repositories/CertificateRepositoryTest.java

					
				@ -20,9 +22,15 @@ class CertificateRepositoryTest {

				    @Autowired

				    private CertificateExtensionRepository extensionRepository;

				    @BeforeEach

MarkL4YG (Migrated from github.com) commented

2024-11-23 10:42:25 +00:00

Also das Hibernate-Problem scheint zu sein, dass er die BLOB-Felder über ResultSet#getBlob lesen will.
Die Methode ist vom SQLite Driver aber nicht implementiert und wirft daher einen Fehler.

Also das Hibernate-Problem scheint zu sein, dass er die BLOB-Felder über ``ResultSet#getBlob`` lesen will. Die Methode ist vom SQLite Driver aber nicht implementiert und wirft daher einen Fehler.