Re: [BUG] [PATCH] Add yank-media and DND handler [9.6.7 (9.6.7-g6eb773 @ /home/viz/lib/emacs/straight/build/org/)]

[Max Nikulin]

On 27/09/2023 15:29, Visuwesh wrote:
> Ihor Radchenko wrote:
> > We can just say :safe nil (omit the keyword). Then, users will be >> prompted and can decide which directories are truly safe for them.> 
That would be quite annoying IMO.  I say we let the user shoot> 
themselves in the foot.
The patch implements 2 features:
- attach to an Org file,
- save to arbitrary directory.

Second option is not specific to Org, so I would consider it in the 
context of Emacs core or an independent ELPA package. I am unsure if 
this is a bright idea, but it eliminates most of questions.

Concerning security issues, there is demand like

> I wish to open Org files by using EWW.
https://debbugs.gnu.org/cgi/bugreport.cgi?bug=58774#34

So a file from net may have arbitrary local variables.

On 27/09/2023 15:33, Visuwesh wrote:
> Max Nikulin wrote:
> > Does it mean that `org--dnd-local-file-handler' is unconditionally
> > called for Org buffers? Current action is to open text files in the
> > widow under cursor and it is what users may expect. They may be
> > surprised if the file is attached instead.
>
> The common request I see when a file is dropped is to associate it
> somehow with the org-mode buffer.  org-attach is the most natural way to
> do it.  If the users like the old behaviour, they can remove the entry
> from the alist.

I am unsure concerning opt-in vs. opt-out choice. I do not follow 
discussions e.g. on emacs help and on reddit, so I can not estimate if 
voice of users happy with opening file and discovered changed defaults 
would be louder.