add comments, fix user expiry log spam

now actually removes the already deleted user from the expiry list.

css/base.css

@@ -52,6 +52,11 @@
     padding: var(--spacing-4,1rem);
 }
 
+.modal-content .banner {
+    margin-left: calc(-1 * var(--spacing-4,1rem) - 0.5%); /* Not sure why this is necessary */
+    margin-right: calc(-1 * var(--spacing-4,1rem) - 0.5%);
+}
+
 div.card:contains(section.banner.footer) {
     padding-bottom: 0px;
 }

html/admin.html

@@ -40,8 +40,8 @@
         </div>
         <div id="modal-about" class="modal">
             <div class="modal-content content card">
-                <span class="heading">{{ .strings.aboutProgram }} <span class="modal-close">&times;</span></span>
+                <img src="{{ .urlBase }}/banner.svg" class="banner header" alt="jfa-go banner">
-                <img src="{{ .urlBase }}/banner.svg" class="mt-1" alt="jfa-go banner">
+                <span class="heading"><span class="modal-close">&times;</span></span>
                 <p><i class="icon ri-github-fill"></i><a href="https://github.com/hrfee/jfa-go">jfa-go</a></p>
                 <p>{{ .strings.version }} <span class="code monospace">{{ .version }}</span></p>
                 <p>{{ .strings.commitNoun }} <span class="code monospace">{{ .commit }}</span></p>

main.go

@@ -79,7 +79,7 @@ type appContext struct {
 	configBase     settings
 	dataPath       string
 	webFS          httpFS
-	cssClass       string
+	cssClass       string // Default theme, "light-theme"|"dark-theme".
 	jellyfinLogin  bool
 	users          []User
 	invalidTokens  []string

mediabrowser/mediabrowser.go

@@ -1,3 +1,6 @@
+// Mediabrowser provides user-related bindings to the Jellyfin & Emby APIs.
+// Some data aren't bound to structs as jfa-go doesn't need to interact with them, for example DisplayPreferences.
+// See Jellyfin/Emby swagger docs for more info on them.
 package mediabrowser
 
 import (
@@ -21,7 +24,8 @@ const (
 	EmbyServer
 )
 
-type serverInfo struct {
+// ServerInfo stores info about the server.
+type ServerInfo struct {
 	LocalAddress string `json:"LocalAddress"`
 	Name         string `json:"ServerName"`
 	Version      string `json:"Version"`
@@ -39,7 +43,7 @@ type MediaBrowser struct {
 	useragent      string
 	auth           string
 	header         map[string]string
-	ServerInfo     serverInfo
+	ServerInfo     ServerInfo
 	Username       string
 	password       string
 	Authenticated  bool
@@ -56,7 +60,7 @@ type MediaBrowser struct {
 	timeoutHandler common.TimeoutHandler
 }
 
-// NewServer returns a new Jellyfin object.
+// NewServer returns a new Mediabrowser object.
 func NewServer(st serverType, server, client, version, device, deviceID string, timeoutHandler common.TimeoutHandler, cacheTimeout int) (*MediaBrowser, error) {
 	mb := &MediaBrowser{}
 	mb.serverType = st

mediabrowser/models.go

@@ -11,6 +11,7 @@ type magicParse struct {
 	Parsed time.Time `json:"parseme"`
 }
 
+// Time embeds time.Time with a custom JSON Unmarshal method to work with Jellyfin & Emby's time formatting.
 type Time struct {
 	time.Time
 }
@@ -62,7 +63,8 @@ type User struct {
 	LastLoginDate             Time          `json:"LastLoginDate"`
 	LastActivityDate          Time          `json:"LastActivityDate"`
 	Configuration             Configuration `json:"Configuration"`
-	Policy                    Policy        `json:"Policy"`
+	// Policy stores the user's permissions.
+	Policy Policy `json:"Policy"`
 }
 
 type SessionInfo struct {

userdaemon.go

@@ -76,6 +76,7 @@ func (app *appContext) checkUsers() {
 			}
 			if !found {
 				app.info.Printf("Expired user already deleted, ignoring.")
+				delete(app.storage.users, id)
 				continue
 			}
 			app.info.Printf("%s expired user \"%s\"", termPlural, user.Name)