Merge pull request #12 from Klemek/dev

v1.2.1
Updated version
2019-06-26 19:35:29 +02:00 · 2019-06-26 19:28:29 +02:00 · 2019-06-26 19:28:18 +02:00 · 2019-06-26 19:28:00 +02:00 · 2019-06-26 19:00:30 +02:00 · 2019-06-26 18:56:01 +02:00
12 changed files with 102 additions and 60 deletions
@@ -3,6 +3,7 @@
 /config.json
 /config.example.json
 /data
 /data/*
 /test_data
 /access.log
 /error.log
@@ -125,6 +125,30 @@ Resources are located on the `data` folder and can be referenced as the root of
 /styles/main.css => data/styles/main.css
 ```
 In your template, the following data is sent :
 * `info`  (every pages)
  * `title` :  the blog's title as in the config
  * `description` the blog's description as in the config
  * `host` : the specified or guessed host with the protocol
  * `version` : the GitBlog.md current running version
  * `request` : the Express request object
  * `config` : the content of the config
 * `article` (article pages only)
  * `title` : the full title
  * `thumbnail` the URL path of the thumbnail
  * `url` : the URL path for this article (with the title)
  * `date` : a JS date
  * `year`
  * `month`
  * `day`
  * `path` : the URL path for the folder of the article (without the title)
  * `realPath` : the system's path for the folder
  * `escapedTitle` : the code with alphanumeric and underscore characters only
 * `error` (error pages only)
  * `error` : the error code
  * `path` : the resource that caused the error
 #### 5. Create and init your git source
 You need to [create a new repository](https://github.com/new) on your favorite Git service.
@@ -132,7 +156,10 @@ You need to [create a new repository](https://github.com/new) on your favorite G
 ```bash
 #gitblog.md/
 cd data
 git init
 git remote add origin <url_of_your_repo.git>
 git add .
 git commit -m "initial commit"
 git push -u origin master
 ```
@@ -157,7 +184,7 @@ Here are the steps for Github, if you use another platform adapt it your way (he
 ```json
 "webhook": {
    "endpoint": "/webhook",
-    "secret": "sha1=<value>",
+    "secret": "<value>",
    "signature_header": "X-Hub-Signature"
  },
 ```
@@ -212,6 +239,9 @@ Any URL like `/year/month/day/anything/` will redirect to this article (and link
 *  `node_port` (default: 3000)  
  the port the server is listening to
 *  `host` (default: none)  
  if set (like `https://mywebsite.com`, it will be used as reference for creating links  
  by default, host is guessed based on first request
 * `data_dir` (default: data)  
  the directory where will be located the git repo with templates and articles
 * `view_engine` (default: ejs)  
@@ -1,6 +1,6 @@
 {
  "name": "gitblog.md",
-  "version": "1.1.4",
+  "version": "1.2.1",
  "description": "A static blog using Markdown pulled from your git repository.",
  "main": "src/server.js",
  "dependencies": {
@@ -3,17 +3,7 @@
 <head>
    <meta charset="UTF-8">
    <title><%= info.title %> - Error <%= error %></title>
-
+    <%- include('head'); %>
    <meta name="twitter:card" content="summary_large_image">
    <%= `<meta property="og:title" content="${info.title} - Home">` %>
    <%= `<meta property="twitter:title" content="${info.title} - Home">` %>
    <%= `<meta property="og:description" content="${info.description}">` %>
    <%= `<meta property="twitter:description" content="${info.description}">` %>
    <%= `<meta property="org:url" content="${info.host}/">` %>
    <meta name="viewport" content="width=device-width, initial-scale=1.0, shrink-to-fit=no">
    <link rel="stylesheet" type="text/css" href="/style.css">
 </head>
 <body>
 <main>
@@ -1,6 +1,6 @@
 <hr>
 <footer>
-    <small><a href="/rss">RSS feed</a> - @<%= new Date().getFullYear() %> - Made with <a
+    <small><a href="/rss">RSS feed</a> - <%= new Date().getFullYear() %> - Made with <a
                href="https://github.com/klemek/gitblog.md">GitBlog.md</a> (v<%= info.version %>)
    </small>
 </footer>
@@ -0,0 +1,25 @@
 <meta name="twitter:card" content="summary_large_image">
 <% if(locals.article){ %>
    <%- `<meta property="og:title" content="${info.title} - ${article.title}">` %>
    <%- `<meta property="twitter:title" content="${info.title} - ${article.title}">` %>
    <%- `<meta property="og:description" content="${info.description}">` %>
    <%- `<meta property="twitter:description" content="${info.description}">` %>
    <%- `<meta property="org:url" content="${info.host + article.url}">` %>
    <% if (article.thumbnail) { %>
        <%- `<meta property="og:image" content="${info.host}/${article.thumbnail}">` %>
        <%- `<meta property="twitter:image" content="${info.host}/${article.thumbnail}">` %>
    <% } %>
    <link rel="stylesheet" type="text/css" href="/prism.css">
 <% } else { %>
    <%- `<meta property="og:title" content="${info.title} - Home">` %>
    <%- `<meta property="twitter:title" content="${info.title} - Home">` %>
    <%- `<meta property="og:description" content="${info.description}">` %>
    <%- `<meta property="twitter:description" content="${info.description}">` %>
    <%- `<meta property="org:url" content="${info.host}/">` %>
 <% } %>
 <link rel="alternate" type="application/rss+xml" title="RSS feed" href="/rss"/>
 <meta name="viewport" content="width=device-width, initial-scale=1.0, shrink-to-fit=no">
 <link rel="stylesheet" type="text/css" href="/style.css">
@@ -3,17 +3,7 @@
 <head>
    <meta charset="UTF-8">
    <title><%= info.title %> - Home</title>
-
+    <%- include('head'); %>
    <meta name="twitter:card" content="summary_large_image">
    <%= `<meta property="og:title" content="${info.title} - Home">` %>
    <%= `<meta property="twitter:title" content="${info.title} - Home">` %>
    <%= `<meta property="og:description" content="${info.description}">` %>
    <%= `<meta property="twitter:description" content="${info.description}">` %>
    <%= `<meta property="org:url" content="${info.host}/">` %>
    <meta name="viewport" content="width=device-width, initial-scale=1.0, shrink-to-fit=no">
    <link rel="stylesheet" type="text/css" href="style.css">
 </head>
 <body>
 <main>
@@ -8,7 +8,7 @@ body, html {
 }
 body {
-    font: 14px/1.45 -apple-system, BlinkMacSystemFont, Segoe UI, sans-serif;
+    font: 15px sans-serif;
    color: #111;
    -webkit-text-size-adjust: none;
    background-color: #F5F5F5;
@@ -16,8 +16,8 @@ body {
 }
 main {
-    max-width: 75ch;
+    max-width: 42rem;
-    padding: 2ch;
+    padding: 2rem;
    margin: auto;
    background-color: #F0F0F0;
    min-height: 100vh;
@@ -58,7 +58,7 @@ blockquote {
    border-left: 0.5em solid #ccc;
    padding-left: 1em;
    margin: 0.25em 0;
-    color: #333;
+    color: #555;
 }
 blockquote > p {
@@ -108,7 +108,7 @@ main.article div.header a.link-home {
    line-height: 2.4;
 }
-main.article div.header h1, main.article div.header h2, div.article h3 {
+main.article div.header h1, main.article div.header h2 {
    margin-top: 0.85em;
    margin-bottom: 0.25em;
    font-size: 2em;
@@ -123,11 +123,19 @@ main.article div.header span.time, div.article span.time {
 }
 div.article {
-    margin-left: 1em;
+    margin: 0 1em 1em 1em;
 }
 div.article h3 {
    font-size: 1.3em;
    margin:0;
 }
 div.article img{
    max-width: 100%;
    height: auto;
    margin-right:1em;
    margin-top:0.25em;
 }
 #text {
@@ -3,22 +3,7 @@
 <head>
    <meta charset="UTF-8">
    <title><%= info.title %> - <%= article.title %></title>
-
+    <%- include('head'); %>
    <meta name="twitter:card" content="summary_large_image">
    <%= `<meta property="og:title" content="${info.title} - ${article.title}">` %>
    <%= `<meta property="twitter:title" content="${info.title} - ${article.title}">` %>
    <%= `<meta property="og:description" content="${info.description}">` %>
    <%= `<meta property="twitter:description" content="${info.description}">` %>
    <%= `<meta property="org:url" content="${info.host + article.url}">` %>
    <% if (article.thumbnail) { %>
        <%= `<meta property="org:image" content="${info.host + article.thumbnail}">` %>
        <%= `<meta property="twitter:image" content="${info.host + article.thumbnail}">` %>
    <% } %>
    <meta name="viewport" content="width=device-width, initial-scale=1.0, shrink-to-fit=no">
    <link rel="stylesheet" type="text/css" href="/prism.css">
    <link rel="stylesheet" type="text/css" href="/style.css">
 </head>
 <body>
 <main class="article">
@@ -36,7 +36,7 @@ module.exports = (config) => {
  const articles = {};
  let lastRSS = '';
-  let host;
+  let host = config['host'];
  /**
   * Fetch articles from the data folder and send success as a response
@@ -67,17 +67,20 @@ module.exports = (config) => {
  /**
   * Render the page with the view engine and catch errors
   * @param req
   * @param res
   * @param vPath - path of the view
   * @param data - data to pass to the view
   * @param code - code to send along the page
   */
-  const render = (res, vPath, data, code = 200) => {
+  const render = (req, res, vPath, data, code = 200) => {
    data.info = {
      title: config['home']['title'],
      description: config['home']['description'],
      host: host,
-      version: pjson.version
+      version: pjson.version,
      request: req,
      config: config
    };
    res.render(vPath, data, (err, html) => {
      if (err) {
@@ -90,17 +93,18 @@ module.exports = (config) => {
  /**
   * Show an error with the correct page
   * @param req
   * @param resPath - the page of the original error
   * @param code - error code
   * @param res
   */
-  const showError = (resPath, code, res) => {
+  const showError = (req, resPath, code, res) => {
    const errorPath = path.join(config['data_dir'], config['home']['error']);
    fs.access(errorPath, fs.constants.R_OK, (err) => {
      if (err)
        res.sendStatus(code);
      else
-        render(res, errorPath, {error: code, path: resPath}, code);
+        render(req, res, errorPath, {error: code, path: resPath}, code);
    });
  };
@@ -135,7 +139,7 @@ module.exports = (config) => {
      if (err)
        showError(req.path, 404, res);
      else
-        render(res, homePath, {articles: Object.values(articles).sort((a, b) => ('' + b.path).localeCompare(a.path))});
+        render(req, res, homePath, {articles: Object.values(articles).sort((a, b) => ('' + b.path).localeCompare(a.path))});
    });
  });
@@ -160,7 +164,7 @@ module.exports = (config) => {
          });
        lastRSS = feed.xml();
      }
-      res.type('rss').send(lastRSS);
+      res.type(req.headers['user-agent'].match(/Mozilla/) ? 'text/xml' : 'rss').send(lastRSS);
    } else {
      showError(req.path, 404, res);
    }
@@ -219,7 +223,7 @@ module.exports = (config) => {
              console.log(cons.error, `no template found at ${templatePath}`);
              showError(req.path, 500, res);
            } else
-              render(res, templatePath, {article: article});
+              render(req, res, templatePath, {article: article});
          });
        });
      }
@@ -1,5 +1,6 @@
 {
  "node_port": 3000,
  "host": "",
  "data_dir": "data",
  "view_engine": "ejs",
  "access_log": "access.log",
@@ -37,7 +38,7 @@
    "endpoint": "/webhook",
    "secret": "",
    "signature_header": "",
-    "pull_command": "git pull"
+    "pull_command": "git pull origin master"
  },
  "showdown": {
    "parseImgDimensions": true,
@@ -175,11 +175,19 @@ describe('Test RSS feed', () => {
  test('200 empty rss', (done) => {
    request(app).get('/rsstest').then((response) => {
      expect(response.statusCode).toBe(200);
      expect(response.type).toBe('application/rss+xml');
      expect(response.text.length).toBeGreaterThan(0);
      expect(response.text.split('<item>').length).toBe(1);
      done();
    });
  });
  test('200 Mozilla fix', (done) => {
    request(app).get('/rsstest').set('user-agent', 'Mozilla Firefox 64.0').then((response) => {
      expect(response.statusCode).toBe(200);
      expect(response.type).toBe('text/xml');
      done();
    });
  });
  test('200 rss cache', (done) => {
    request(app).get('/rsstest').then(() => {
      request(app).get('/rsstest').then((response) => {
Author	SHA1	Message	Date
Klemek	4397a76d9b	Merge pull request #12 from Klemek/dev v1.2.1	2019-06-26 19:35:29 +02:00
Klemek	ddf964eb27	Updated version	2019-06-26 19:28:29 +02:00
Klemek	4b47276484	Updated readme	2019-06-26 19:28:18 +02:00
Klemek	a7fedb149f	Host from config if specified	2019-06-26 19:28:00 +02:00
Klemek	ea95a285c9	Merge pull request #11 from Klemek/dev v1.2.0	2019-06-26 19:00:30 +02:00
Klemek	0fde428806	Updated coverage	2019-06-26 18:56:01 +02:00
Klemek	8fc7ff1ca7	Updated version	2019-06-26 18:44:31 +02:00
Klemek	ae4e2eb8d5	Fixing Firefox RSS handling	2019-06-26 18:43:41 +02:00
Klemek	528e4be1fe	Updated templates	2019-06-26 18:34:40 +02:00
Klemek	bd42883330	Update template.ejs	2019-06-26 18:21:44 +02:00
Klemek	b6ac0a73b4	Update style.css	2019-06-26 18:21:26 +02:00
Klemek	aebc3da5bc	Update template.ejs	2019-06-26 16:43:03 +02:00
Klemek	7a4a4f9006	Update footer.ejs	2019-06-26 15:07:32 +02:00
Klemek	1341aa5a56	Update config.default.json	2019-06-26 11:46:06 +02:00
Klemek	5e05f250f4	Update style.css	2019-06-26 10:06:12 +02:00
Klemek	6cf7be3afb	Update README.md	2019-06-23 15:37:23 +02:00
Klemek	6aceacad18	Update README.md	2019-06-23 15:34:46 +02:00
Klemek	a3a23be1c2	Update README.md	2019-06-23 15:34:34 +02:00
Klemek	e8e8024021	Merge pull request #10 from Klemek/dev v1.1.5	2019-06-23 15:15:45 +02:00
Klemek	1806d60ca7	Fixed meta tags being wrong	2019-06-23 15:14:47 +02:00