cowboy: anchor emojy pages to URLs

This also refactored a bunch of junk and eliminated about 4k calls to
the config/database on emojy listing.
This commit is contained in:
Chris Sexton 2022-07-22 09:44:42 -04:00
parent 79cdd0f828
commit cf19a2bc15
9 changed files with 278 additions and 58 deletions

View File

@ -118,7 +118,11 @@ func New(config *config.Config, connector Connector) Bot {
log.Debug().Msgf("created web router")
// Make the http logger optional
// It has never served a purpose in production and with the emojy page, can make a rather noisy log
if bot.Config().GetInt("bot.useLogger", 0) == 1 {
bot.router.Use(middleware.Logger)
}
bot.router.Use(middleware.StripSlashes)
bot.router.HandleFunc("/", bot.serveRoot)

View File

@ -12,12 +12,19 @@ import (
type Cowboy struct {
b bot.Bot
c *config.Config
emojyPath string
baseEmojyURL string
}
func New(b bot.Bot) *Cowboy {
emojyPath := b.Config().Get("emojy.path", "emojy")
baseURL := b.Config().Get("emojy.baseURL", "/emojy/file")
c := Cowboy{
b: b,
c: b.Config(),
emojyPath: emojyPath,
baseEmojyURL: baseURL,
}
c.register()
c.registerWeb()

View File

@ -15,8 +15,8 @@ import (
"github.com/velour/catbase/plugins/emojy"
)
func getEmojy(c *config.Config, name string) (image.Image, error) {
files, _, err := emojy.AllFiles(c)
func getEmojy(emojyPath, baseEmojyURL, name string) (image.Image, error) {
files, _, err := emojy.AllFiles(emojyPath, baseEmojyURL)
if err != nil {
return nil, err
}
@ -35,8 +35,7 @@ func getEmojy(c *config.Config, name string) (image.Image, error) {
return img, nil
}
func getCowboyHat(c *config.Config) (image.Image, error) {
emojyPath := c.Get("emojy.path", "emojy")
func getCowboyHat(c *config.Config, emojyPath string) (image.Image, error) {
p := path.Join(emojyPath, c.Get("cowboy.hatname", "hat.png"))
p = path.Clean(p)
f, err := os.Open(p)
@ -50,8 +49,8 @@ func getCowboyHat(c *config.Config) (image.Image, error) {
return img, nil
}
func cowboyifyImage(c *config.Config, input image.Image) (image.Image, error) {
hat, err := getCowboyHat(c)
func cowboyifyImage(c *config.Config, emojyPath string, input image.Image) (image.Image, error) {
hat, err := getCowboyHat(c, emojyPath)
if err != nil {
return nil, err
}
@ -66,12 +65,12 @@ func cowboyifyImage(c *config.Config, input image.Image) (image.Image, error) {
return dst, nil
}
func cowboy(c *config.Config, name string) ([]byte, error) {
emojy, err := getEmojy(c, name)
func cowboy(c *config.Config, emojyPath, baseEmojyURL, name string) ([]byte, error) {
emojy, err := getEmojy(emojyPath, baseEmojyURL, name)
if err != nil {
return nil, err
}
img, err := cowboyifyImage(c, emojy)
img, err := cowboyifyImage(c, emojyPath, emojy)
if err != nil {
return nil, err
}

View File

@ -15,7 +15,7 @@ func (p *Cowboy) registerWeb() {
func (p *Cowboy) handleImage(w http.ResponseWriter, r *http.Request) {
what := chi.URLParam(r, "what")
img, err := cowboy(p.c, what)
img, err := cowboy(p.c, p.emojyPath, p.baseEmojyURL, what)
if err != nil {
w.WriteHeader(500)
fmt.Fprintf(w, "Error: %s", err)

View File

@ -26,16 +26,23 @@ type EmojyPlugin struct {
b bot.Bot
c *config.Config
db *sqlx.DB
emojyPath string
baseURL string
}
const maxLen = 32
func New(b bot.Bot) *EmojyPlugin {
log.Debug().Msgf("emojy.New")
emojyPath := b.Config().Get("emojy.path", "emojy")
baseURL := b.Config().Get("emojy.baseURL", "/emojy/file")
p := &EmojyPlugin{
b: b,
c: b.Config(),
db: b.DB(),
emojyPath: emojyPath,
baseURL: baseURL,
}
p.setupDB()
p.register()
@ -193,11 +200,9 @@ func (p *EmojyPlugin) allCounts() (map[string][]EmojyCount, error) {
return out, nil
}
func AllFiles(c *config.Config) (map[string]string, map[string]string, error) {
func AllFiles(emojyPath, baseURL string) (map[string]string, map[string]string, error) {
files := map[string]string{}
urls := map[string]string{}
emojyPath := c.Get("emojy.path", "emojy")
baseURL := c.Get("emojy.baseURL", "/emojy/file")
entries, err := os.ReadDir(emojyPath)
if err != nil {
return nil, nil, err
@ -214,7 +219,7 @@ func AllFiles(c *config.Config) (map[string]string, map[string]string, error) {
}
func (p *EmojyPlugin) isKnownEmojy(name string) (bool, string, string, error) {
allFiles, allURLs, err := AllFiles(p.c)
allFiles, allURLs, err := AllFiles(p.emojyPath, p.baseURL)
if err != nil {
return false, "", "", err
}

106
plugins/emojy/list.html Normal file
View File

@ -0,0 +1,106 @@
<!DOCTYPE html>
<html lang="en">
<head>
<!-- Load required Bootstrap and BootstrapVue CSS -->
<link type="text/css" rel="stylesheet" href="//unpkg.com/bootstrap/dist/css/bootstrap.min.css"/>
<link type="text/css" rel="stylesheet" href="//unpkg.com/bootstrap-vue@^2/dist/bootstrap-vue.min.css"/>
<!-- Load polyfills to support older browsers -->
<script src="//polyfill.io/v3/polyfill.min.js?features=es2015%2CMutationObserver"></script>
<!-- Load Vue followed by BootstrapVue -->
<script src="//unpkg.com/vue@^2/dist/vue.min.js"></script>
<script src="//unpkg.com/bootstrap-vue@^2/dist/bootstrap-vue.min.js"></script>
<script src="https://unpkg.com/vue-router@^2"></script>
<script src="https://unpkg.com/axios/dist/axios.min.js"></script>
<meta charset="UTF-8">
<title>Memes</title>
</head>
<body>
<div id="app">
<b-navbar>
<b-navbar-brand>Emojys</b-navbar-brand>
<b-navbar-nav>
<b-nav-item v-for="item in nav" :href="item.url" :active="item.name === 'Meme'" :key="item.key">{{ item.name
}}
</b-nav-item>
</b-navbar-nav>
</b-navbar>
<b-navbar>
<b-navbar-nav>
<b-nav-item href="/emojy/stats">Stats</b-nav-item>
<b-nav-item active href="/emojy/list">List</b-nav-item>
<b-nav-item href="/emojy/upload">Upload</b-nav-item>
</b-navbar-nav>
</b-navbar>
<div
style="background-color:red;"
variant="error"
v-if="err != ''"
@click="err = ''">
{{ err }}
</div>
<ul>
<li v-for="(path, name) in fileList" key="name">
{{name}}:
<img :src="path" :alt="name" class="img-thumbnail" style="max-width: 64px; max-height: 64px"/>
</li>
</ul>
</div>
<script>
var router = new VueRouter({
mode: 'history',
routes: []
});
var app = new Vue({
el: '#app',
router,
data: function () {
return {
err: '',
view: '',
nav: [],
results: [],
fileList: {},
image: null,
password: ''
}
},
watch: {
view(newView, oldView) {
this.err = '';
}
},
mounted() {
axios.get('/nav')
.then(resp => {
this.nav = resp.data;
})
.catch(err => console.log(err))
this.refresh();
},
methods: {
refresh: function () {
axios.get('/emojy/all')
.then(resp => {
this.results = resp.data
this.err = ''
})
.catch(err => (this.err = err))
axios.get('/emojy/allFiles')
.then(resp => {
this.fileList = resp.data
this.err = ''
})
.catch(err => (this.err = err))
},
}
})
</script>
</body>
</html>

118
plugins/emojy/stats.html Normal file
View File

@ -0,0 +1,118 @@
<!DOCTYPE html>
<html lang="en">
<head>
<!-- Load required Bootstrap and BootstrapVue CSS -->
<link type="text/css" rel="stylesheet" href="//unpkg.com/bootstrap/dist/css/bootstrap.min.css"/>
<link type="text/css" rel="stylesheet" href="//unpkg.com/bootstrap-vue@^2/dist/bootstrap-vue.min.css"/>
<!-- Load polyfills to support older browsers -->
<script src="//polyfill.io/v3/polyfill.min.js?features=es2015%2CMutationObserver"></script>
<!-- Load Vue followed by BootstrapVue -->
<script src="//unpkg.com/vue@^2/dist/vue.min.js"></script>
<script src="//unpkg.com/bootstrap-vue@^2/dist/bootstrap-vue.min.js"></script>
<script src="https://unpkg.com/vue-router@^2"></script>
<script src="https://unpkg.com/axios/dist/axios.min.js"></script>
<meta charset="UTF-8">
<title>Memes</title>
</head>
<body>
<div id="app">
<b-navbar>
<b-navbar-brand>Emojys</b-navbar-brand>
<b-navbar-nav>
<b-nav-item v-for="item in nav" :href="item.url" :active="item.name === 'Meme'" :key="item.key">{{ item.name
}}
</b-nav-item>
</b-navbar-nav>
</b-navbar>
<b-navbar>
<b-navbar-nav>
<b-nav-item active href="/emojy/stats">Stats</b-nav-item>
<b-nav-item href="/emojy/list">List</b-nav-item>
<b-nav-item href="/emojy/upload">Upload</b-nav-item>
</b-navbar-nav>
</b-navbar>
<div
style="background-color:red;"
variant="error"
v-if="err != ''"
@click="err = ''">
{{ err }}
</div>
<ul>
<li v-for="(category, name) in results" key="name">
{{name}}:
<ul>
<li v-for="emojy in category" key="emojy">
{{emojy.count}} -
<span v-if="name != 'emoji'">
<span v-if="emojy.onServer"></span>
<span v-else></span>
-
</span>
<img v-if="emojy.url" :src="emojy.url" :alt="emojy.name" class="img-thumbnail"
style="max-width: 64px; max-height: 64px"/>
<span v-else>{{emojy.emojy}}</span>
</li>
</ul>
</li>
</ul>
</div>
<script>
var router = new VueRouter({
mode: 'history',
routes: []
});
var app = new Vue({
el: '#app',
router,
data: function () {
return {
err: '',
view: '',
nav: [],
results: [],
fileList: {},
image: null,
password: ''
}
},
watch: {
view(newView, oldView) {
this.err = '';
}
},
mounted() {
axios.get('/nav')
.then(resp => {
this.nav = resp.data;
})
.catch(err => console.log(err))
this.refresh();
},
methods: {
refresh: function () {
axios.get('/emojy/all')
.then(resp => {
this.results = resp.data
this.err = ''
})
.catch(err => (this.err = err))
axios.get('/emojy/allFiles')
.then(resp => {
this.fileList = resp.data
this.err = ''
})
.catch(err => (this.err = err))
},
}
})
</script>
</body>
</html>

View File

@ -30,9 +30,9 @@
<b-navbar>
<b-navbar-nav>
<b-nav-item :active="view == 'stats'" @click="view = 'stats'">Stats</b-nav-item>
<b-nav-item :active="view == 'list'" @click="view = 'list'">List</b-nav-item>
<b-nav-item :active="view == 'upload'" @click="view = 'upload'">Upload</b-nav-item>
<b-nav-item href="/emojy/stats">Stats</b-nav-item>
<b-nav-item href="/emojy/list">List</b-nav-item>
<b-nav-item active href="/emojy/upload">Upload</b-nav-item>
</b-navbar-nav>
</b-navbar>
@ -44,7 +44,7 @@
{{ err }}
</div>
<div class="container" v-if="view == 'upload'">
<div class="container">
<label>Passphrase</label>
<b-input v-model="password"></b-input>
<label>File
@ -53,32 +53,6 @@
<br>
<button @click="submitFile()">Submit</button>
</div>
<ul v-if="view == 'stats'">
<li v-for="(category, name) in results" key="name">
{{name}}:
<ul>
<li v-for="emojy in category" key="emojy">
{{emojy.count}} -
<span v-if="name != 'emoji'">
<span v-if="emojy.onServer"></span>
<span v-else></span>
-
</span>
<img v-if="emojy.url" :src="emojy.url" :alt="emojy.name" class="img-thumbnail"
style="max-width: 64px; max-height: 64px"/>
<span v-else>{{emojy.emojy}}</span>
</li>
</ul>
</li>
</ul>
<ul v-if="view == 'list'">
<li v-for="(path, name) in fileList" key="name">
{{name}}:
<img :src="path" :alt="name" class="img-thumbnail" style="max-width: 64px; max-height: 64px"/>
</li>
</ul>
</div>
<script>

View File

@ -25,13 +25,22 @@ func (p *EmojyPlugin) registerWeb() {
r.HandleFunc("/allFiles", p.handleAllFiles)
r.HandleFunc("/upload", p.handleUpload)
r.HandleFunc("/file/{name}", p.handleEmojy)
r.HandleFunc("/stats", p.handlePage("stats.html"))
r.HandleFunc("/list", p.handlePage("list.html"))
r.HandleFunc("/upload", p.handlePage("upload.html"))
r.HandleFunc("/", p.handleIndex)
p.b.RegisterWebName(r, "/emojy", "Emojys")
}
func (p *EmojyPlugin) handleIndex(w http.ResponseWriter, r *http.Request) {
index, _ := embeddedFS.ReadFile("index.html")
http.Redirect(w, r, "/emojy/stats", http.StatusPermanentRedirect)
}
func (p *EmojyPlugin) handlePage(file string) func(w http.ResponseWriter, r *http.Request) {
return func(w http.ResponseWriter, r *http.Request) {
index, _ := embeddedFS.ReadFile(file)
w.Write(index)
}
}
func (p *EmojyPlugin) handleAll(w http.ResponseWriter, r *http.Request) {
@ -47,7 +56,7 @@ func (p *EmojyPlugin) handleAll(w http.ResponseWriter, r *http.Request) {
}
func (p *EmojyPlugin) handleAllFiles(w http.ResponseWriter, r *http.Request) {
_, urlMap, err := AllFiles(p.c)
_, urlMap, err := AllFiles(p.emojyPath, p.baseURL)
if err != nil {
w.WriteHeader(500)
out, _ := json.Marshal(struct{ err error }{err})
@ -107,13 +116,12 @@ func (p *EmojyPlugin) FileSave(r *http.Request) (string, error) {
if ok, _, _, _ := p.isKnownEmojy(emojyName); ok {
return "", fmt.Errorf("emojy already exists")
}
emojyPath := p.c.Get("emojy.path", "emojy")
contentType := fileHeader.Header.Get("Content-Type")
if !strings.HasPrefix(contentType, "image") {
return "", fmt.Errorf("incorrect mime type - given: %s", contentType)
}
fullPath := filepath.Clean(filepath.Join(emojyPath, emojyFileName))
_ = os.MkdirAll(emojyPath, os.ModePerm)
fullPath := filepath.Clean(filepath.Join(p.emojyPath, emojyFileName))
_ = os.MkdirAll(p.emojyPath, os.ModePerm)
log.Debug().Msgf("trying to create/open file: %s", fullPath)
file, err := os.OpenFile(fullPath, os.O_WRONLY|os.O_CREATE, os.ModePerm)
if err != nil {
@ -132,8 +140,7 @@ func (p *EmojyPlugin) FileSave(r *http.Request) (string, error) {
func (p *EmojyPlugin) handleEmojy(w http.ResponseWriter, r *http.Request) {
fname := chi.URLParam(r, "name")
emojyPath := p.c.Get("emojy.path", "emojy")
contents, err := ioutil.ReadFile(path.Join(emojyPath, fname))
contents, err := ioutil.ReadFile(path.Join(p.emojyPath, fname))
if err != nil {
w.WriteHeader(404)
out, _ := json.Marshal(struct{ err error }{err})