Attachment Improvements By Xon 1.1.3

[Xon]

Xon submitted a new resource:

Attachment Improvements By Xon - A collection of improvements to XF's attachment system

A collection of improvements to XF's attachment system.

• SVG support
• Nginx's X-Accel-Redirect
• New Permissions for forum/conversations (Respects global attachment size & count limits): -- Attachment Size (kb). -- Maximum Attachment Count.

As XenForo_FileOutput is not easily extensible, this add-on effectively reimplements:

• XenForo_ViewAdmin_Attachment_View
• XenForo_ViewPublic_Attachment_View

SVG Support
Support for SVG attachments to be...

Read more about this resource...

 

[Xon]

Xon updated Attachment Improvements By Xon with a new update entry:

Feature update release

Automatically uninstall old 'SVG Attachments' and 'XAR Attachments' add-ons.

Read the rest of this update entry...

 

[silence]

Are there side effects of uninstalling the addon after usage?

 

[Xon]

Are there side effects of uninstalling the addon after usage?

No., it will revert to standard XenForo behaviour if this add-on is disabled or uninstalled.

But oops, I forgot to make it delete the permissions it leaves behind I'll fix that tomorrow.

 

[BobbyWibowo]

When you upload a new SVG file and then embed it to the post as a thumbnail, it'll appear as full instead of thumbnail on the editor, but will display correctly as a thumbnail after you post it. Was that intentional?

 

[Xon]

When you upload a new SVG file and then embed it to the post as a thumbnail, it'll appear as full instead of thumbnail on the editor, but will display correctly as a thumbnail after you post it. Was that intentional?

That sounds like a bug, but it is a while since I've touched the SVG attachment aspect. I'll need to have a look at it again.

 

[BobbyWibowo]

My members aren't particularly using SVG files, so there have been no issue. I simply happened to encounter that issue while I was testing out things after I switched to this add-on from the nginx-accel only one.

 

[Xon]

My members aren't particularly using SVG files, so there have been no issue. I simply happened to encounter that issue while I was testing out things after I switched to this add-on from the nginx-accel only one.

Looking into this more, and the entire thumbnail handling is broken. I've got a fix just under going testing and should be released in a day or two.

Next version will support thumbnail rebuilding and on install will kick off a rebuild of any SVG attachments.

 

[Xon]

Xon updated Attachment Improvements By Xon with a new update entry:

Bugfix release

• Cleanup permissions on uninstall.
• Set SVG thumbnail dimensions correctly.
• Support SVG attachment thumbnail resizing.

Note; on installing this version the SVG Attachment Thumbnail resize operation will be started to correct any existing SVG attachments thumbnails to respect the thumbnail attachment dimensions.

Read the rest of this update entry...

 

[BobbyWibowo]

@Xon Got error after uploading an SVG file:

ErrorException: Fatal Error: Call to undefined method AttachmentViewer_XenForo_Model_Attachment::extractDimensionsForSVG() -library/SV/AttachmentImprovements/Deferred/SVGAttachmentThumb.php:35

 

[Axel B]

Hi m8,
I use the previous addon for SVG attachments. My website doesn't run on NGINX. Can I still use this addon?

 

[BobbyWibowo]

@Axel B I'm pretty sure you can. You only need an nginx server if you want to enable the nginx-accel feature thing, which is actually disabled by default. Since it's disabled by default, I assume it's so that non-nginx users can use the other features.

 

[Xon]

Hi m8,
I use the previous addon for SVG attachments. My website doesn't run on NGINX. Can I still use this addon?

The nginx-only parts are disabled by default.

@Xon Got error after uploading an SVG file:

ErrorException: Fatal Error: Call to undefined method AttachmentViewer_XenForo_Model_Attachment::extractDimensionsForSVG() -library/SV/AttachmentImprovements/Deferred/SVGAttachmentThumb.php:35

Did you by any chance have the add-on disabled when installed? Or switch it to disabled after installing?

This error happens when the deferred task runs but the add-on isn't enabled. I'll add some detection code so the deferred task exits out earlier rather than errors.

 

[Axel B]

Thanks for the confirmation guys

 

[BobbyWibowo]

@Xon For some reason, SVG is still not being treated as an image. Whenever I upload a new SVG file, it'll be treated as a normal file, so there's no option to add thumbnail
This is the SVG file that I tried to upload: https://upload.wikimedia.org/wikipedia/en/a/a4/Flag_of_the_United_States.svg
Also tried another one (UK flag), didn't work as well. There's no error on the error log by the way.

 

[Xon]

@Xon For some reason, SVG is still not being treated as an image. Whenever I upload a new SVG file, it'll be treated as a normal file, so there's no option to add thumbnail
This is the SVG file that I tried to upload: https://upload.wikimedia.org/wikipedia/en/a/a4/Flag_of_the_United_States.svg
Also tried another one (UK flag), didn't work as well. There's no error on the error log by the way.

How it extracts the width/height is a bit fragile, I'll need to-do some tweaking.

:edit: Figured it out, I'm scanning the SVG for known-bad tags (such as <use xlink:href="..." />which can pull in external resources) and it is tripping over an internal reference which should be allowed.

 

[Xon]

@BobbyWibowo after re-reading the SVG spec and some security presentations; removing the 'use' statement from the "Bad SVG tags" should be safe. Especially as the SVG file is being loading as an image which browser use to enforce a number of restrictions.

I'll soon release a version which gives an error message when an SVG is rejected.

 

[BobbyWibowo]

@Xon Alright, thanks!

 

[Xon]

Xon updated Attachment Improvements By Xon with a new update entry:

Maintenance update

• Improve width/height extraction
• (optional) Rejection of invalid SVG with an error message.
• Support for applying SVG restrictions to previous attachments via the deferred task.
• Remove 'use' tag from the bad tag list (for now).
• Workaround for attachment eTag being mangled into a weak eTag (Cloudflare has been seen to-do this).

Read the rest of this update entry...

 

[kontrabass]

Hi Xon!, found this addon whilst looking for SVG support, and it is working great! Thanks!

Regarding the X-Accel-Redirect functionality: am I correct in guessing that this would have the effect of freeing up resources of the webserver? I'm just a bit confused as to what the benefit actually is. And then to implement, we would just copy your 'location' directive suggestion for 'internal data in webroot', to our nginx vhost config file?

We have a ton of attachments and are running the latest nginx.

Thanks!!