~emersion

France

https://emersion.fr

~emersion/alps-dev

Last active 17 hours ago

~emersion/public-inbox

Last active 2 days ago

~emersion/soju-dev

Last active 17 days ago

~emersion/mrsh-dev

Last active 27 days ago
View more

Recent activity

Re: [PATCH v2] Set Message-Id header 17 hours ago

From Simon Ser to ~emersion/alps-dev

On Wednesday, August 5, 2020 8:10 PM, <j3s@c3f.net> wrote:

> August 5, 2020 8:01 AM, "Simon Ser" contact@emersion.fr wrote:
>
> > Thanks! Pushed with a minor edit to change indentation to tabs instead
> > of spaces.
>
> Derp! I'll be sure to gofmt all future patches.
>
> Thanks for helping turn this around so quickly. It sounds like you were
> interested in keeping other Message-ID's around if they had already existed,
> is that work you're interested in having done? I'd be happy to take a stab at it.

Yes, of course! I think it'll involve adding a MessageID field to

Re: [PATCH v2] Set Message-Id header 5 days ago

From Simon Ser to ~emersion/alps-dev

Thanks! Pushed with a minor edit to change indentation to tabs instead
of spaces.

On Wednesday, August 5, 2020 1:54 AM, Jesse Olson <j3s@c3f.net> wrote:

> ---
>
> > go-message already has a helper to generate Message-Id header fields
>
> Oh woops, silly me! Let me know if this patch looks good, it overwrites
> the Message-Id header as requested!
>
> The only thing I noticed is that the header is defined as "Message-Id"
> rather than "Message-ID", not a big deal but I don't think it's 100%

Re: Session timeouts discard drafts 6 days ago

From Simon Ser to ~emersion/alps-dev

Hi,

On Tuesday, August 4, 2020 6:05 AM, <j3s@c3f.net> wrote:

> Currently, session timeouts have the potential to cause unexpected email loss.
>
> Recently, one of our users was composing an email and pressed the "save to drafts"
> button after her session had already timed out. After pressing this button her
> carefully composed email was discarded and she was presented with a login screen.
>
> I'd like to talk about ideas for improving this situation. My immediate thought
> was automatically saving emails that are being composed into Drafts every N seconds,
> which seems like the obvious solution. But I'm not sure if that jives with what
> you'd like to do.

Re: [PATCH] Set Message-Id header 6 days ago

From Simon Ser to ~emersion/alps-dev

> Hi! I have personal interest in getting this patch upstream because it is biting a few of my users.

Hi! Thanks for your patch!

> This is a proof of concept Message-ID implementation, mostly for discussion.
>
> The format is partially gleaned from https://talk.begriffs.com/pipermail/friends/2020-August/001144.html
>
> This patch doesn't work exactly as provided, but is a rough draft of the idea - I'm sure you'd like to see
> generateMessageID moved to a different place, for example, but I wasn't sure where. So, here are my questions.
>
> todo: currently the domain isn't stripped from the from addr, but it should be trivial to do.
>
> 0. do you want have plans for this or am i stepping on toes here?

Re: Getting "Invalid charset specified" 7 days ago

From Simon Ser to ~sircmpwn/aerc

Hmm, I don't find this string in go-message, nor in the Go standard
library...

Re: [PATCH gamja v4] Add error reporting on connect and main page 10 days ago

From Simon Ser to ~emersion/public-inbox

Thanks for this new version! Looks a lot better, just a few remaining
issues and it'll be good to go :)

On Monday, July 27, 2020 9:17 PM, bbworld1 <vwangsf@gmail.com> wrote:

> ---
>
> The unrelated changes should be gone. Apologies.
> I accidentally sent another email without the gamja tag in the header.
> The patches are identical, but please disregard that one.
>
>  components/app.js     | 22 ++++++++++++++++------
>  components/connect.js |  1 +
>  lib/client.js         | 11 +++++------

Re: Variable substitution for the exec command 11 days ago

From Simon Ser to ~sircmpwn/aerc

On Tuesday, July 28, 2020 11:52 AM, Nicolai Dagestad <nicolai@dagestad.fr> wrote:

> Hi,
> I'm adding a simple variable substitution for the exec command so I can sync my
> maildirs with a shortcut from aerc.
> What would be the prefered format, something like `{acount}` and `{folder}`
> or %a and %f.

These solutions would have the downside of not allowing proper quoting.
If the folder contains a space or a quote, I don't think it's possible
to come up with a solution that works.

An alternative would be to set env variables and let the shell perform
the expansion. So it would be something like $account and $folder.

Re: [PATCH v3] Add error reporting on connect and main page 14 days ago

From Simon Ser to ~emersion/public-inbox

Thanks for sending this new version, however it still seems like you'll
need to send a new one! :P

This patch includes unrelated changes, I think the changes are coming
from recent commits pushed to master. Maybe this can help:

    git fetch
    # Delete the broken commit, but keeps the local changes:
    git reset origin/master
    # And then create a new commit as usual

See [1] for a good resource about editing Git history.

[1]: https://git-rebase.io/