Hi,

Just spent some time writing a little custom script for a NodeBB based photography forum, its job was to show some EXIF data in a small box under uploaded photos. (e.g. camera model).

Updated to latest version, and poof.. all EXIF data is stripped...
https://github.com/NodeBB/NodeBB/commit/5eaa14e746f3526138ba2433029d70a1870980bc

I get that it is a sane default, since EXIF data can be somewhat sensitive from a privacy standpoint, however is there any chance of making this configurable?

I can get by with hand editing the code after each update but that is a pain...

I can't seem to find any trigger that is triggered before the EXIF data is stripped... Any ideas?

0 barisusakli committed to NodeBB/NodeBB

fix: strip exif data on image uploads

Hi,
I am trying to use the static hook: static:messaging.canMessageUser to add custom logic to the permission checks if a user may message another user.
However, it seems that static hooks cannot alter the flow of nodebb. Throwing an error or returning false etc. do not do anything.
Shouldn't this rather be a filter hook?
Or am I misunderstanding something?

@Chris1 you can see some limited stats over here

When a user has read privilege on a category but not on the topics within the category, the teaser still shows.
Is that intentional?

Fixed already!!

Thanks @baris !

When a user has read privilege on a category but not on the topics within the category, the teaser still shows.
Is that intentional?

Hi,
I am trying to use the static hook: static:messaging.canMessageUser to add custom logic to the permission checks if a user may message another user.
However, it seems that static hooks cannot alter the flow of nodebb. Throwing an error or returning false etc. do not do anything.
Shouldn't this rather be a filter hook?
Or am I misunderstanding something?

Fixed already!!

Thanks @baris !

Hi,

Just spent some time writing a little custom script for a NodeBB based photography forum, its job was to show some EXIF data in a small box under uploaded photos. (e.g. camera model).

Updated to latest version, and poof.. all EXIF data is stripped...
https://github.com/NodeBB/NodeBB/commit/5eaa14e746f3526138ba2433029d70a1870980bc

I get that it is a sane default, since EXIF data can be somewhat sensitive from a privacy standpoint, however is there any chance of making this configurable?

I can get by with hand editing the code after each update but that is a pain...

I can't seem to find any trigger that is triggered before the EXIF data is stripped... Any ideas?

0 barisusakli committed to NodeBB/NodeBB

fix: strip exif data on image uploads

I am wondering if this forum is maintained in sync with current master? or is it only updated on a version release?
( @julian, you fixed https://github.com/NodeBB/NodeBB/issues/7593 yesterday, wondering when can I test the fix on the current forum? I was actually pretty sure that it was an issue in frontend script as I attempted to explain in the issue... )

yossizahn created this issue in NodeBB/NodeBB

closed Can't add password when logged in via Google without a password #7593

@Chris1 you can see some limited stats over here