Correctly encode the audit log header to a URI component (#659)

* Correctly encode the audit log header to a URI component * Add test for encoding unicode audit log reason --------- Co-authored-by: Mylo Fawcett <[email protected]>
nyxx-discord · May 23, 2024 · 27b74b5 · 27b74b5
1 parent 7908853
commit 27b74b5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/src/http/request.dart b/lib/src/http/request.dart
@@ -74,7 +74,7 @@ abstract class HttpRequest {
 
   Map<String, String> _getHeaders(Nyxx client) => {
         userAgent: client.apiOptions.userAgent,
-        if (auditLogReason != null) xAuditLogReason: auditLogReason!,
+        if (auditLogReason != null) xAuditLogReason: Uri.encodeComponent(auditLogReason!),
         if (authenticated) authorization: client.apiOptions.authorizationHeader,
         ...headers,
       };

diff --git a/test/integration/rest_integration_test.dart b/test/integration/rest_integration_test.dart
@@ -275,7 +275,7 @@ void main() {
       );
 
       await expectLater(
-        webhook.delete(),
+        webhook.delete(auditLogReason: 'Testing Unicode in audit log reason 😀'),
         completes,
       );
     });