Fixed panic on mapping columns to settings #676

Zhaars · 2023-10-04T11:57:59Z

Checklist

Change is covered by automated tests
The coding guidelines are followed
Public API has proper documentation in the Acra documentation site or has PR on documentation repository
with new changes
CHANGELOG.md is updated (in case of notable or breaking changes)
CHANGELOG_DEV.md is updated
Benchmark results are attached (if applicable)
Example projects and code samples are up-to-date (in case of API changes)

vixentael · 2023-10-04T12:06:39Z

decryptor/postgresql/pg_decryptor.go

@@ -1026,7 +1026,7 @@ func (proxy *PgProxy) handleQueryDataPacket(ctx context.Context, packet *PacketH
 			format = int(boundFormat)
 		}
 		var encryptionSetting config.ColumnEncryptionSetting = nil
-		if encryptionSettings != nil && i <= len(encryptionSettings) && encryptionSettings[i] != nil {
+		if encryptionSettings != nil && i < len(encryptionSettings) && encryptionSettings[i] != nil {


Fixed panic on mapping columns to settings

fc0ed03

Zhaars requested a review from Lagovas October 4, 2023 11:57

vixentael reviewed Oct 4, 2023

View reviewed changes

Lagovas approved these changes Oct 4, 2023

View reviewed changes

Zhaars merged commit 1194aa8 into master Oct 5, 2023
11 checks passed

Zhaars deleted the zhars/fixed_panic_on_mapping_settings_to_columns branch September 12, 2024 17:10

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fixed panic on mapping columns to settings #676

Fixed panic on mapping columns to settings #676

Zhaars commented Oct 4, 2023

vixentael Oct 4, 2023

Fixed panic on mapping columns to settings #676

Fixed panic on mapping columns to settings #676

Conversation

Zhaars commented Oct 4, 2023

Checklist

vixentael Oct 4, 2023

Choose a reason for hiding this comment