Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

NIL is not of type CHARACTER (w/ commit b5ae82b) #38

Open
cdmojoli opened this issue Jan 16, 2025 · 0 comments
Open

NIL is not of type CHARACTER (w/ commit b5ae82b) #38

cdmojoli opened this issue Jan 16, 2025 · 0 comments

Comments

@cdmojoli
Copy link

Commit b5ae82b modifies the creation of request objects introducing add-query-with-input, which effectively changes the type of the :path slot from string to QURI.URI:URI.

This triggers an error inside QURI::MAYBE-SLASH, where (char/= (uiop:first-char path) #\/) expects a char but instead gets (UIOP:FIRST-CHAR path) → nil.

To illustrate:

The value
  NIL
is not of type
  CHARACTER
BREAK was entered because of *BREAK-ON-SIGNALS* (now rebound to NIL).
   [Condition of type SIMPLE-CONDITION]

...

Backtrace:
 0: ((FLET QURI::MAYBE-SLASH :IN QURI:RENDER-URI) "dynamodb.us-east-1.amazonaws.com" #<QURI.URI:URI />)
 1: (QURI:RENDER-URI #<error printing URI-HTTPS {1005B00573}> NIL)
 2: (AWS-SDK/API:AWS-REQUEST #<AWS-SDK/SERVICES/DYNAMODB/API:DYNAMODB-REQUEST {1005927583}> :WANT-STREAM NIL)
 3: (AWS-SDK/SERVICES/DYNAMODB/API:GET-ITEM :RETURN-CONSUMED-CAPACITY "TOTAL" :TABLE-NAME "*****" :KEY #<HASH-TABLE :TEST EQL :COUNT 1 {1005ACA3C3}>)

Just changing the result type of add-query-with-input to string should solve this, but I do not know the programmer's intent here, so it's just a suggestion.

@cdmojoli cdmojoli changed the title NIL is not of type CHARACTER (w/ commit b5ae82) NIL is not of type CHARACTER (w/ commit b5ae82b) Jan 16, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant