Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add filter to allow modification of blob container properties #11

Closed
wants to merge 4 commits into from
Closed

Add filter to allow modification of blob container properties #11

wants to merge 4 commits into from

Conversation

640-KB
Copy link
Contributor

@640-KB 640-KB commented Nov 15, 2016

Create a filter called 'azure_blob_put_blob_properties' which enables modification of Azure blob storage properties.

For example, to add HTTP Cache-Control of 1 week, add the following to your theme, etc:

add_filter( 'azure_blob_put_blob_properties', function ( $headers ) {
    $headers['x-ms-blob-cache-control'] = 'public, max-age=604800';
    return $headers;
} );

Source:
https://wordpress.org/support/topic/headers-for-uploaded-blobs/#post-8440264

@Ritesh-patel
Copy link
Contributor

Hi @mrgregwaugh

Thanks for the PR. If you can rename filter to windows_azure_storage_blob_properties, that will be great.

@Ritesh-patel Ritesh-patel self-assigned this Nov 16, 2016
@640-KB
Copy link
Contributor Author

640-KB commented Nov 16, 2016

Done. Had to bump due to issues with github this morning not updating commit.

@Ritesh-patel
Copy link
Contributor

Hey @mrgregwaugh

I forgot to mention that you need to create PR for develop branch, can you open a new PR for that? I am closing this one.

Ritesh-patel pushed a commit that referenced this pull request Nov 17, 2016
Marked old functions as deprecated, used new methods wherever possible
@jperalesHogarth jperalesHogarth mentioned this pull request May 30, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants