internetstandards / Internet.nl

Internet standards compliance test suite
https://internet.nl
164 stars 36 forks source link

Need some further help with forked site customization #1341

Closed apio-sys closed 2 weeks ago

apio-sys commented 3 months ago

I have the language part working as per the updated documentation/Customize.md a part from a small issue with the integration-test as posted separately here #1338 .

I still have quite a few questions to continue the customization of a fork of the project. You mentioned that others are currently working on custom sites. Maybe we can all share any findings and tips & tricks. I'll do my best to help update documentation where I can.

For the language part cont'd:

<input id="internetnl-site-url" type="url" name="url" placeholder="www.domaine.fr">

that change is not picked-up on the front after rebuilding. I'm not sure how to integrate such changes (and if it's wise to do so since touching core files...).

Styling:

security.txt:

    # disable security.txt if branding is disabled
    set $internetnl_branding "False";
    location = /.well-known/security.txt {
      if ($internetnl_branding != "True"){
        return 404;
      }
      # basic auth should not apply to this path
      auth_basic off;
      # IP allowlist should also not apply
      allow all;
      alias /var/www/internet.nl/.well-known/security.txt;
    }

Should this just be harcoded to the wanted value (and updated content in security.txt)?

bwbroersma commented 2 months ago

Thanks for the valid questions.

About example.nl, that should indeed be a variable, I'm a bit unsure if it should be a in the translation or an environment variable.

The security.txt setup will probably change a bit, related:

Related:

apio-sys commented 1 month ago

Thanks for the reply @bwbroersma and the reference to the discussion in #1188 . Even though it is still a discussion, I would appreciate some indications to start and get the ball rolling. I.e. get the news page working (this doesn't seem to pickup from the translated version in the fork) and add an about page. Should I go for direct modifications inside the /interface/templates folder or will this pose an issue keeping the fork in sync if things change upstream? Just looking for the cleanest method while keeping the fork in sync all the time. Sometimes this poses problem i.e. with the certbot.sh script since subdomain names for languagues are hardcoded, but that is manageable by cherrypicking during merges. But it shouldn't be a cause to get of the rails completely and ending up with a fork that doesn't have it's core up2date.

mxsasha commented 1 month ago

We don't have a specific recommendation for how to modify the templates I think. Currently we barely change them at all. We are working towards a redesign which will change many templates, but that's pretty long term. Underway is rewriting the TLS code, but I don't think you'll change that much.

apio-sys commented 2 weeks ago

I think we can now close this as per the resolution in #1418 which works fine. Thanks!