Wonderful!
@mbcahyono could you please create a new topic for the plugin in plugin?
Wonderful!
@mbcahyono could you please create a new topic for the plugin in plugin?
The Discourse Images Guardian plugin no longer works with the latest Discourse beta ā you canāt set a URL for the logos any more so all images now require authentication, this results in the site logos returning 404ās for non-authenticated users, I have opened an issue for this on GitHub to see if we can find a way to solve this.
The Discourse Images Guardian has been updated by @mbcahyono so it now works with the latest Discourse beta, I have deployed it to a couple of servers running v2.3.0.beta1 .
The Discourse Images Guardian plugin doesnāt currently work with the latest Discourse v2.3.0.beta5, I have had to disable it on a couple of sites. I have raised an issue for this on GitHub and hopefully @mbcahyono will have a suggestion for a fix for this soon.
Today we began internally trialling a new āsecure mediaā setting, which is only usable if S3 uploads are enabled. What this will do is the following:
This setting is currently only available for self-hosted Discourse instances. We will provide further updates once we have completed internal trials.
What are the downsides of this? Does it effect performance at all?
The URL for media in posts/private messages is simply replaced with a Discourse server URL which then serves the private S3 URL for the upload.
Just to clarify here.
The downside is that you can not use a CDN anymore for private stuff and every image request / download has to āpipeā through the app to unlock the bucket for a user.
On small internal forums it is very unlikely you will notice anything. On a huge forums that require login, giving up the CDN and piping requests via the app may have some impact.
The Image Guardian plugin doesnāt work with the latest Discourse, Iāve opened an issue for this at GitHub:
I did look at using the Secure uploads option but that requires S3 to be used and on the forum in question this isnāt the case. So it is still needed and hopefully @mbcahyono is able to solve thisā¦