Added Tags to RSS feed as categories

cmd/data/articles.go

@@ -21,15 +21,17 @@ type Article struct {
 	AuthorID int64
 }
 
+// TODO: setCh
 type ArticleList struct {
 	addCh    chan *Article
 	delCh    chan uuid.UUID
-	retCh    chan *Article
 	getCh    chan []Article
+	retCh    chan *Article
 	articles []*Article
 	wg       sync.WaitGroup
 }
 
+// TODO: setCh
 type TagList struct {
 	addCh chan string
 	getCh chan []string
@@ -37,16 +39,16 @@ type TagList struct {
 	wg    sync.WaitGroup
 }
 
-func minArticleList() *ArticleList {
+func initArticleList() *ArticleList {
 	return &ArticleList{
 		addCh: make(chan *Article),
 		delCh: make(chan uuid.UUID),
-		retCh: make(chan *Article),
 		getCh: make(chan []Article),
+		retCh: make(chan *Article),
 	}
 }
 
-func minTagList() *TagList {
+func initTagList() *TagList {
 	return &TagList{
 		addCh: make(chan string),
 		getCh: make(chan []string),
@@ -89,8 +91,8 @@ func (tl *TagList) start() {
 }
 
 func NewArticleList() *ArticleList {
-	list := minArticleList()
+	list := initArticleList()
-	list.articles = []*Article{}
+	list.articles = make([]*Article, 0)
 
 	list.wg.Add(1)
 	go list.start()
@@ -106,6 +108,7 @@ func (al *ArticleList) Add(a *Article) {
 func (al *ArticleList) Release(uuid uuid.UUID) (*Article, bool) {
 	al.delCh <- uuid
 	article := <-al.retCh
+
 	if article == nil {
 		return nil, false
 	}
@@ -123,10 +126,8 @@ func (al *ArticleList) Save(filename string) error {
 	}
 	defer file.Close()
 
-	encoder := gob.NewEncoder(file)
 	articles := al.Get()
-	err = encoder.Encode(articles)
+	if err = gob.NewEncoder(file).Encode(articles); err != nil {
-	if err != nil {
 		return fmt.Errorf("error ecoding key: %v", err)
 	}
 
@@ -139,10 +140,8 @@ func LoadArticleList(filename string) (*ArticleList, error) {
 		return nil, fmt.Errorf("error opening key file: %v", err)
 	}
 
-	decoder := gob.NewDecoder(file)
 	articleList := NewArticleList()
-	err = decoder.Decode(&articleList.articles)
+	if err = gob.NewDecoder(file).Decode(&articleList.articles); err != nil {
-	if err != nil {
 		return nil, fmt.Errorf("error decoding key: %v", err)
 	}
 
@@ -150,8 +149,8 @@ func LoadArticleList(filename string) (*ArticleList, error) {
 }
 
 func NewTagList() *TagList {
-	list := minTagList()
+	list := initTagList()
-	list.tags = []string{}
+	list.tags = make([]string, 0)
 
 	list.wg.Add(1)
 	go list.start()
@@ -175,10 +174,8 @@ func (tl *TagList) Save(filename string) error {
 	}
 	defer file.Close()
 
-	encoder := gob.NewEncoder(file)
 	tags := tl.Get()
-	err = encoder.Encode(tags)
+	if err = gob.NewEncoder(file).Encode(tags); err != nil {
-	if err != nil {
 		return fmt.Errorf("error ecoding key: %v", err)
 	}
 
@@ -190,11 +187,10 @@ func LoadTagList(filename string) (*TagList, error) {
 	if err != nil {
 		return nil, fmt.Errorf("error opening key file: %v", err)
 	}
+	defer file.Close()
 
-	decoder := gob.NewDecoder(file)
 	tagList := NewTagList()
-	err = decoder.Decode(&tagList.tags)
+	if err = gob.NewDecoder(file).Decode(&tagList.tags); err != nil {
-	if err != nil {
 		return nil, fmt.Errorf("error decoding key: %v", err)
 	}
 

cmd/data/db.go

@@ -45,8 +45,7 @@ func (db *DB) AddUser(user *User, pass string) error {
         (username, password, first_name, last_name, role)
     VALUES (?, ?, ?, ?, ?)
     `
-	_, err = db.Exec(query, user.UserName, string(hashedPass), user.FirstName, user.LastName, user.Role)
+	if _, err = db.Exec(query, user.UserName, string(hashedPass), user.FirstName, user.LastName, user.Role); err != nil {
-	if err != nil {
 		return fmt.Errorf("error inserting user into DB: %v", err)
 	}
 
@@ -104,8 +103,7 @@ func (db *DB) ChangePassword(id int64, oldPass, newPass string) error {
     SET password = ?
     WHERE id = ?
     `
-	_, err = db.Exec(query, string(newHashedPass), id)
+	if _, err = db.Exec(query, string(newHashedPass), id); err != nil {
-	if err != nil {
 		return fmt.Errorf("error updating password in DB: %v", err)
 	}
 
@@ -134,8 +132,7 @@ func (db *DB) GetUser(id int64) (*User, error) {
     `
 
 	row := db.QueryRow(query, id)
-	if err := row.Scan(&user.ID, &user.UserName, &user.FirstName,
+	if err := row.Scan(&user.ID, &user.UserName, &user.FirstName, &user.LastName, &user.Role); err != nil {
-		&user.LastName, &user.Role); err != nil {
 		return nil, fmt.Errorf("error reading user information: %v", err)
 	}
 

cmd/data/rss.go

@@ -6,101 +6,99 @@ import (
 	"os"
 	"sync"
 
-	"github.com/gorilla/feeds"
+	"git.streifling.com/jason/rss"
 )
 
-type Feed struct {
+type Channel struct {
-	addCh chan *feeds.Item
+	addCh   chan *rss.Item
-	setCh chan feeds.Feed
+	setCh   chan rss.Channel
-	getCh chan feeds.Feed
+	getCh   chan rss.Channel
-	feed  feeds.Feed
+	channel rss.Channel
 	wg      sync.WaitGroup
 }
 
-func minFeed() *Feed {
+func initChannel() *Channel {
-	return &Feed{
+	return &Channel{
-		addCh: make(chan *feeds.Item),
+		addCh: make(chan *rss.Item),
-		setCh: make(chan feeds.Feed),
+		setCh: make(chan rss.Channel),
-		getCh: make(chan feeds.Feed),
+		getCh: make(chan rss.Channel),
+		channel: rss.Channel{
+			Items: make([]*rss.Item, 0),
+		},
 	}
 }
 
-func (f *Feed) start() {
+func (c *Channel) start() {
-	f.wg.Done()
+	c.wg.Done()
 	for {
 		select {
-		case item := <-f.addCh:
+		case item := <-c.addCh:
-			f.feed.Items = append(f.feed.Items, item)
+			c.channel.Items = append(c.channel.Items, item)
-		case f.getCh <- f.feed:
+		case c.getCh <- c.channel:
-		case f.feed = <-f.setCh:
+		case c.channel = <-c.setCh:
 		}
 	}
 }
 
-func NewFeed(title, link, desc string) *Feed {
+func NewChannel(title, link, desc string) *Channel {
-	feed := minFeed()
+	channel := initChannel()
-	feed.feed = feeds.Feed{
+	channel.channel = rss.Channel{
 		Title:       title,
-		Link:        &feeds.Link{Href: link},
+		Link:        link,
 		Description: desc,
 	}
 
-	feed.wg.Add(1)
+	channel.wg.Add(1)
-	go feed.start()
+	go channel.start()
-	feed.wg.Wait()
+	channel.wg.Wait()
 
-	return feed
+	return channel
 }
 
-func (f *Feed) Get() feeds.Feed {
+func (c *Channel) Get() rss.Channel {
-	return <-f.getCh
+	return <-c.getCh
 }
 
-func (f *Feed) Set(feed feeds.Feed) {
+func (f *Channel) Set(channel rss.Channel) {
-	f.setCh <- feed
+	f.setCh <- channel
 }
 
-func OpenFeed(filename string) (*Feed, error) {
+func LoadChannel(filename string) (*Channel, error) {
 	file, err := os.Open(filename)
 	if err != nil {
 		return nil, fmt.Errorf("error opening file %v: %v", filename, err)
 	}
 	defer file.Close()
 
-	feed := minFeed()
+	channel := initChannel()
-	feed.wg.Add(1)
+	channel.wg.Add(1)
-	go feed.start()
+	go channel.start()
-	feed.wg.Wait()
+	channel.wg.Wait()
 
-	decoder := gob.NewDecoder(file)
+	tmpChannel := new(rss.Channel)
-	tmpFeed := new(feeds.Feed)
+	if err = gob.NewDecoder(file).Decode(tmpChannel); err != nil {
-	err = decoder.Decode(tmpFeed)
+		return nil, fmt.Errorf("error decoding channel from file %v: %v", filename, err)
-	if err != nil {
-		return nil, fmt.Errorf("error decoding file %v: %v", filename, err)
 	}
 
-	feed.Set(*tmpFeed)
+	channel.Set(*tmpChannel)
-
+	return channel, nil
-	return feed, nil
 }
 
-func (f *Feed) Save(filename string) error {
+func (c *Channel) Save(filename string) error {
 	file, err := os.Create(filename)
 	if err != nil {
 		return fmt.Errorf("error creating file %v: %v", filename, err)
 	}
 	defer file.Close()
 
-	encoder := gob.NewEncoder(file)
+	channel := c.Get()
-	feed := f.Get()
+	if err = gob.NewEncoder(file).Encode(channel); err != nil {
-	err = encoder.Encode(feed)
-	if err != nil {
 		return fmt.Errorf("error encoding file %v: %v", filename, err)
 	}
 
 	return nil
 }
 
-func (f *Feed) Add(i *feeds.Item) {
+func (c *Channel) Add(i *rss.Item) {
-	f.addCh <- i
+	c.addCh <- i
 }

cmd/data/sessions.go

@@ -17,8 +17,7 @@ type CookieStore struct {
 func NewKey() ([]byte, error) {
 	key := make([]byte, 32)
 
-	_, err := io.ReadFull(rand.Reader, key)
+	if _, err := io.ReadFull(rand.Reader, key); err != nil {
-	if err != nil {
 		return nil, fmt.Errorf("error generating key: %v", err)
 	}
 
@@ -33,9 +32,7 @@ func SaveKey(key []byte, filename string) error {
 	defer file.Close()
 	file.Chmod(0600)
 
-	encoder := gob.NewEncoder(file)
+	if err = gob.NewEncoder(file).Encode(key); err != nil {
-	err = encoder.Encode(key)
-	if err != nil {
 		return fmt.Errorf("error ecoding key: %v", err)
 	}
 
@@ -49,9 +46,7 @@ func LoadKey(filename string) ([]byte, error) {
 	}
 
 	key := make([]byte, 32)
-	decoder := gob.NewDecoder(file)
+	if err = gob.NewDecoder(file).Decode(&key); err != nil {
-	err = decoder.Decode(&key)
-	if err != nil {
 		return nil, fmt.Errorf("error decoding key: %v", err)
 	}
 

cmd/ui/articles.go

@@ -6,8 +6,8 @@ import (
 	"net/http"
 	"time"
 
+	"git.streifling.com/jason/rss"
 	"github.com/google/uuid"
-	"github.com/gorilla/feeds"
 	"streifling.com/jason/cpolis/cmd/data"
 )
 
@@ -58,6 +58,9 @@ func FinishArticle(al *data.ArticleList, s *data.CookieStore) http.HandlerFunc {
 			return
 		}
 
+		r.ParseForm()
+		article.Tags = append(article.Tags, r.Form["tags"]...)
+
 		session, err := s.Get(r, "cookie")
 		if err != nil {
 			tmpl, err := template.ParseFiles("web/templates/login.html")
@@ -71,7 +74,7 @@ func FinishArticle(al *data.ArticleList, s *data.CookieStore) http.HandlerFunc {
 		article.AuthorID = session.Values["id"].(int64)
 
 		al.Add(article)
-		al.Save("tmp/articles.gob")
+		al.Save("tmp/unpublished-articles.gob")
 
 		tmpl, err := template.ParseFiles("web/templates/hub.html")
 		tmpl = template.Must(tmpl, err)
@@ -116,7 +119,7 @@ func ReviewArticle(al *data.ArticleList, s *data.CookieStore) http.HandlerFunc {
 	}
 }
 
-func PublishArticle(f *data.Feed, al *data.ArticleList, s *data.CookieStore) http.HandlerFunc {
+func PublishArticle(c *data.Channel, al *data.ArticleList, s *data.CookieStore) http.HandlerFunc {
 	return func(w http.ResponseWriter, r *http.Request) {
 		uuid, err := uuid.Parse(r.PostFormValue("uuid"))
 		if err != nil {
@@ -141,14 +144,15 @@ func PublishArticle(f *data.Feed, al *data.ArticleList, s *data.CookieStore) htt
 			template.Must(tmpl, err).ExecuteTemplate(w, "page-content", msg)
 		}
 
-		f.Add(&feeds.Item{
+		c.Add(&rss.Item{
 			Title:       article.Title,
-			Author:      &feeds.Author{Name: session.Values["name"].(string)},
+			Author:      session.Values["name"].(string),
-			Created:     article.Created,
+			PubDate:     article.Created.Format(time.RFC1123Z),
 			Description: article.Desc,
-			Content:     article.Content,
+			Content:     &rss.Content{Value: article.Content},
+			Categories:  article.Tags,
 		})
-		f.Save("tmp/rss.gob")
+		c.Save("tmp/rss.gob")
 
 		tmpl, err := template.ParseFiles("web/templates/hub.html")
 		tmpl = template.Must(tmpl, err)

cmd/ui/rss.go

@@ -5,13 +5,16 @@ import (
 	"log"
 	"net/http"
 
+	"git.streifling.com/jason/rss"
 	"streifling.com/jason/cpolis/cmd/data"
 )
 
-func ShowRSS(f *data.Feed) http.HandlerFunc {
+func ShowRSS(c *data.Channel) http.HandlerFunc {
 	return func(w http.ResponseWriter, r *http.Request) {
-		feed := f.Get()
+		channel := c.Get()
-		rss, err := feed.ToRss()
+		feed := rss.NewFeed()
+		feed.Channels = append(feed.Channels, &channel)
+		rss, err := feed.ToXML()
 		if err != nil {
 			log.Println(err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)

go.mod

@@ -3,6 +3,7 @@ module streifling.com/jason/cpolis
 go 1.22.0
 
 require (
+	git.streifling.com/jason/rss v0.0.0-20240305164907-524bf9676188
 	github.com/go-sql-driver/mysql v1.7.1
 	github.com/google/uuid v1.6.0
 	github.com/gorilla/feeds v1.1.2

go.sum

@@ -1,3 +1,11 @@
+git.streifling.com/jason/rss v0.0.0-20240305152729-9d8cc6464565 h1:/eO9NTksh+9yLz3HiYNR7BJo/iMTxxW5/d9h8I6vR6E=
+git.streifling.com/jason/rss v0.0.0-20240305152729-9d8cc6464565/go.mod h1:gpZF0nZbQSstMpyHD9DTAvlQEG7v4pjO5c7aIMWM4Jg=
+git.streifling.com/jason/rss v0.0.0-20240305160544-c8551159fe32 h1:G25NZzsD73rOkGYgV2vPUDviB0JXk5qi+dXOwB6J56U=
+git.streifling.com/jason/rss v0.0.0-20240305160544-c8551159fe32/go.mod h1:gpZF0nZbQSstMpyHD9DTAvlQEG7v4pjO5c7aIMWM4Jg=
+git.streifling.com/jason/rss v0.0.0-20240305160829-6cd08bb65d2a h1:TWQ9gwe7eWjaLUrZ0CJSc+sUUOw3VoGHlR3F8mH6vqs=
+git.streifling.com/jason/rss v0.0.0-20240305160829-6cd08bb65d2a/go.mod h1:gpZF0nZbQSstMpyHD9DTAvlQEG7v4pjO5c7aIMWM4Jg=
+git.streifling.com/jason/rss v0.0.0-20240305164907-524bf9676188 h1:C8M/j3f+cl5Y7YfGpU/ynb/SC/4tTYMDsyGFt3rswM8=
+git.streifling.com/jason/rss v0.0.0-20240305164907-524bf9676188/go.mod h1:gpZF0nZbQSstMpyHD9DTAvlQEG7v4pjO5c7aIMWM4Jg=
 github.com/aymerick/douceur v0.2.0 h1:Mv+mAeH1Q+n9Fr+oyamOlAkUNPWPlA8PPGR0QAaYuPk=
 github.com/aymerick/douceur v0.2.0/go.mod h1:wlT5vV2O3h55X9m7iVYN0TBM0NH/MmbLnd30/FjWUq4=
 github.com/go-sql-driver/mysql v1.7.1 h1:lUIinVbN1DY0xBg0eMOzmmtGoHwWBbvnWubQUrtU8EI=

main.go

@@ -29,10 +29,10 @@ func main() {
 	}
 	defer db.Close()
 
-	feed, err := data.OpenFeed("tmp/rss.gob")
+	feed, err := data.LoadChannel("tmp/rss.gob")
 	if err != nil {
 		log.Println(err)
-		feed = data.NewFeed("Freimaurer Distrikt Niedersachsen und Sachsen-Anhalt",
+		feed = data.NewChannel("Freimaurer Distrikt Niedersachsen und Sachsen-Anhalt",
 			"https://distrikt-ni-st.de",
 			"Freiheit, Gleichheit, Brüderlichkeit, Toleranz und Humanität")
 	}
@@ -47,7 +47,7 @@ func main() {
 	}
 	store := data.NewCookieStore(key)
 
-	articleList, err := data.LoadArticleList("tmp/articles.gob")
+	articleList, err := data.LoadArticleList("tmp/unpublished-articles.gob")
 	if err != nil {
 		articleList = data.NewArticleList()
 	}