Convert image [Paid] 4.7

[gogo]

I'm just testing XF2.2 on my dev version and this addon still seems to function as intended. Of course a simple reply from the author a couple of months ago should have sufficed.


Your questions aren't really valid for this addon as the intended use for this is to not proxy anything but instead copy the remote image to your Xenforo server as an attachment.

When this add-on is trying to download the linked image from another server, will that server log the real IP address of my server?

 

[dknife]

When this add-on is trying to download the linked image from another server, will that server log the real IP address of my server?

I'd have to say yes since it's the HTTP request from your server that's making the call to the image, not Cloudflare.

 

[gogo]

I'd have to say yes since it's the HTTP request from your server that's making the call to the image, not Cloudflare.

That's why I got those questions regarding IP leaking..

I want to be sure there won't be IP leaking issue when using this app. But how?

 

[gerryvz]

It works on XF 2.2 just fine.

if you go to the developer’s site, you will find all compatibility documented there. And also AndyB will answer any questions you post to that site about the add-ons. He is responsive to questions as well as suggestions and comments.

 

[ZixDev]

Hello, It seems not to work with the ".bmp" file.

 

[rdn]

AndyB updated Convert image with a new update entry:

See description



Read the rest of this update entry...

For Exclude domain, domains need to be separated by comma or space only?
It also accept sub domain right?

Thanks Andy.

 

[rdn]

For Exclude domain, domains need to be separated by comma or space only?
It also accept sub domain right?

Thanks Andy.

When 2 domains is used and separated by comma, it doesn't work .

 

[agison]

Nice addon, here are some fixes you could implement if you want.

1.) Support Webp images -> convert to png automatically. (PHP 7.1+)
2.) Regex fails for some posts like \nURL, the following will match everything.
3.) Urls will sometimes start with a space and end with a space

Thanks again for the awesome addon.

1+ for webp support!

 

[eDaddi]

domains need to be separated by comma or space only

Also looking for how to exclude multiple domains ... like my own forum and giphy for example

 

[AndyB]

AndyB updated Convert image with a new update entry:

See description

Convert image v3.9 changes:

Code now only runs if post message is new or edited.

Read the rest of this update entry...

 

[AndyB]

AndyB updated Convert image with a new update entry:

See description

Convert image v4.0 changes:

Added Extension check to options page.

Read the rest of this update entry...

 

[AndyB]

AndyB updated Convert image with a new update entry:

See description

Convert image v4.1 changes:

Now supports webp images by converting them to jpg.

Read the rest of this update entry...

 

[eDaddi]

@AndyB Is that an option that can be turned off? I'm not sure of a situation that I'd want a optimized webp being replaced by one that takes more storage.

 

[Fullmental]

Hi, we noticed this addon does not preserve image attributes within the [img] tags, for example, if a user sets an image to be sized as such:

[IMG width="100px"][IMG]

Still becomes a full size version of the original image:

[ATTACH=full][/ATTACH]

The expectation is the ATTACH tag would include the original width or height attribute:

[ATTACH=full width="100px"][/ATTACH]

Would it be possible to preserve these image dimensions? Some examples of valid img syntax:

[img][/img]
[img height="20px"][/img]
[img width=10% height=auto][/img]
[IMG height="20px" width="50px"][/img]

Example output:

ATTACH output always the same:

---

Spoiler: configuration issue

The addon is also seemingly not converting img tags nested in a URL tag correctly, for example:

[url=google.com]Next -> [img]https://iili.io/HSkSW1n.png[/img][/url]

The intended result is this, note the text and image are both clickable:

Next ->

But the addon converts it to this, destroying the link information in the process not just for the image, but for the text as well:

Next ->[ATTACH=full]285618[/ATTACH]

Actual result:

Next ->

Note the links being stripped from nested image tags is an option that can be turned off in ACP per Andy. It is apparently enabled by default and we didn't see it.

 

[AndyB]

AndyB updated Convert image with a new update entry:

See description

Convert image v4.2 changes:

Now retains IMG bbcode options.

Read the rest of this update entry...

 

[AndyB]

AndyB updated Convert image with a new update entry:

See description

Convert image v4.3 changes:

Added 'Exclude domains' to options page.

Read the rest of this update entry...

 

[AndyB]

AndyB updated Convert image with a new update entry:

See description

Convert image v4.4 changes:

Added 'Convert conversations' to options page.

Read the rest of this update entry...

 

[AndyB]

AndyB updated Convert image with a new update entry:

See description

Convert image v4.5 changes:

Attachment tables xf_attachment and xf_attachment_data are now updated more accurately to reflect original post date and post user id.

Read the rest of this update entry...

 

[AndyB]

AndyB updated Convert image with a new update entry:

See description

Convert image v4.6 changes:

Fixed issue with converting images in conversations.

Read the rest of this update entry...

 

[AndyB]

AndyB updated Convert image with a new update entry:

See description

Convert image v4.7 changes:

Fixed issue with checking for WEBP support when using GD.

Read the rest of this update entry...