Eason09053360 commented on PR #62224:
URL: https://github.com/apache/airflow/pull/62224#issuecomment-3935915841

   > @Eason09053360, I'd suggest adding some tests to validate that this has 
solved the issue. Based on the issue that was reposted, the strings being 
stored were not > 200 characters. Were you able to reproduce that issue and 
test it?
   
   Thanks for the suggestion! I added a test that shows the root cause.
   
   Even when the raw URL template is < 200 chars, the value stored in DB is the 
signed token, which is much longer. 
   
   Example from the test using a raw template length of 180:
   raw_len :180
   token_len :311
   
   This is why it fails with VARCHAR(200) even though the original string isn’t 
> 200.
   
   Test added: test_sync_bundles_to_db_with_long_signed_url_template.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to