Skip to content

fix: encode PURL qualifiers as a normalized string#466

Merged
behnazh-w merged 1 commit into
stagingfrom
fix-purl-processing
Sep 12, 2023
Merged

fix: encode PURL qualifiers as a normalized string#466
behnazh-w merged 1 commit into
stagingfrom
fix-purl-processing

Conversation

@behnazh-w

@behnazh-w behnazh-w commented Sep 10, 2023

Copy link
Copy Markdown
Member

This PR sets encode=True to encode qualifiers of a PURL string as a normalized string while converting it to a dictionary because SQLite doesn't support dict type.

It also adds exception handling for deserializing a PURL string while initializing a Component instance.

This PR sets ``encode=True`` to encode qualifiers as a normalized
string because SQLite doesn't support ``dict`` type.

It also adds exception handling for deserializing a PURL string while
initializing a Component instance.

Signed-off-by: behnazh-w <behnaz.hassanshahi@oracle.com>
@behnazh-w behnazh-w requested a review from tromai as a code owner September 10, 2023 07:21
@oracle-contributor-agreement oracle-contributor-agreement Bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Sep 10, 2023
Comment thread src/macaron/database/table_definitions.py
@behnazh-w behnazh-w merged commit 4282c6c into staging Sep 12, 2023
@tromai tromai deleted the fix-purl-processing branch September 12, 2023 03:47
art1f1c3R pushed a commit that referenced this pull request Nov 29, 2024
This PR sets `encode=True` to encode qualifiers of a PURL string as a normalized string while converting it to a dictionary and storing it to the SQLite database because SQLite doesn't support dict type.

It also adds exception handling for deserializing a PURL string while initializing a Component instance.

Signed-off-by: behnazh-w <behnaz.hassanshahi@oracle.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants