~xenrox


#205 API: UpdatePatchset does not work for mailing lists of other users 28 days ago

Ticket created by ~xenrox on ~sircmpwn/lists.sr.ht

The UpdatePatchset mutation currently fails if one wants to update a patch from a list where the user does have moderate access but isn't the list owner. This is caused by this query which hard-codes the list owner as the current user.

Instead the access table should be consulted.

#24 pages publish subcommand should support cache-control option 2 months ago

Comment by ~xenrox on ~emersion/hut

There was a small discussion in IRC on how this could look:

<emersion> it's a bit tricky to describe in CLI flags though…
<emersion> maybe --files <glob> --cache-control <value>
<ddevault> --cache 'foo/*.html:cache-control header goes here'?
<emersion> or that
<ddevault> alternatively, --cache=cache.rules where cache.rules is a file with slightly more structure

I would be in favor of the last option. A file should allow the biggest flexibility and leaves room for future settings without adding too many flags. The file could look similar to this - if we want to avoid pulling in new dependencies and use scfg:

notFound "404.html"
fileConfigs {
    glob "*.png" {
        cacheControl "max-age=100"
    }

    glob "*.html" {
        cacheControl "max-age=1"
    }
}

With that we could add a site-config flag to hut and deprecate not-found (or we leave notFound out from the file). Do you have further thoughts on this ~emersion?

#26 Supports build artifacts 2 months ago

on ~emersion/hut

Sorry for the late reply. This looks perfect to me! Thank you very much! “~xenrox” outgoing@sr.ht writes:

Does something along the lines of this patch work for you?

#26 Supports build artifacts 3 months ago

Comment by ~xenrox on ~emersion/hut

Does something along the lines of this patch work for you?

#27 git: add subcommand for setting custom README 3 months ago

Comment by ~xenrox on ~emersion/hut

I have created a simple example repository, that uses a build manifest with the hut development version to set the custom README. Maybe this is enough of a reference point for now.

#27 git: add subcommand for setting custom README 3 months ago

Comment by ~xenrox on ~emersion/hut

hut can already do this. It was added in this commit, which is not released yet. That is why I haven't yet send a patch to update man.sr.ht.

#2 Correctly handle nullable time.Time 5 months ago

Comment by ~xenrox on ~emersion/gqlclient

The time documentation recommends that it should typically not be passed as a pointer and that IsZero can be used to determine uninitialized times. But if there are indeed legitimate use-cases where you would use the zero value, then time should probably be removed from this case. Otherwise the backend has to check with IsZero.

#3 Support interfaces 5 months ago

Ticket created by ~xenrox on ~emersion/gqlclient

Some initial discussion happened in hut-dev

#2 Correctly handle nullable time.Time 5 months ago

Ticket created by ~xenrox on ~emersion/gqlclient

#21 SIGSEGV in "builds list" with invalid access token 9 months ago

Comment by ~xenrox on ~emersion/hut

This bug is fixed with this commit. We actually don't need an additional nil check since the query only uses non-nullable fields.

REPORTED RESOLVED FIXED