Collaborator

Description

• Added name validation for storages in accordance with Apify platform restrictions

Issues

• Closes: Validate storage names before passing them to the storage client #1434
• Relates: All crawlee persistence names should be valid names for Apify platform #1354

Testing

• Added new tests to verify correct validation.

add storages name validation

c3c571b

up docs

40a7227

Collaborator

assigning @janbuchar as @Pijukatel is off for this week

vdusek reviewed Oct 7, 2025

View reviewed changes

Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I'm wondering, shouldn't we add this new enforcement on this level to https://crawlee.dev/python/docs/upgrading/upgrading-to-v1?

janbuchar approved these changes Oct 7, 2025

View reviewed changes

Co-authored-by: Vlada Dusek <v.dusek96@gmail.com>

Co-authored-by: Vlada Dusek <v.dusek96@gmail.com>

Collaborator

@Mantisus sorry did you overlook this:

Thanks. I'm wondering, shouldn't we add this new enforcement on this level to https://crawlee.dev/python/docs/upgrading/upgrading-to-v1?

What do you think about that?

Collaborator Author

Sorry, yes

What do you think about that?

I'm not sure, because it's a fix. And we didn't include fixes in the upgrading guide, if I'm not mistaken.

Collaborator Author

The upgrading guide was updated after discussing on Slack

Cannot create default Dataset when starting Apify actor on local environment apify/apify-sdk-python#648