Skip to content
This repository was archived by the owner on Dec 16, 2022. It is now read-only.

Limit preview for large image files #503

Merged
merged 1 commit into from
Nov 22, 2015
Merged

Limit preview for large image files #503

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions defaults/config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,17 @@ module.exports = {
//
prefetch: false,

//
// Prefetch URLs Image Preview size limit
//
// If prefetch is enabled, Shout will only display content under the maximum size.
// Default value is 512 (in kB)
//
// @type int
// @default 512
//
prefetchMaxImageSize: 512,

//
// Display network
//
Expand Down
15 changes: 13 additions & 2 deletions src/plugins/irc-events/link.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ module.exports = function(irc, network) {
};

function parse(msg, url, res, client) {
var config = Helper.getConfig();
var toggle = msg.toggle = {
id: msg.id,
type: "",
Expand All @@ -61,6 +62,9 @@ function parse(msg, url, res, client) {
link: url
};
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whiteline error


if (!config.prefetchMaxImageSize) {
config.prefetchMaxImageSize = 512;
}
switch (res.type) {
case "text/html":
var $ = cheerio.load(res.text);
Expand All @@ -80,7 +84,12 @@ function parse(msg, url, res, client) {
case "image/gif":
case "image/jpg":
case "image/jpeg":
toggle.type = "image";
if (res.size < (config.prefetchMaxImageSize * 1024)) {
toggle.type = "image";
}
else {
return;
}
break;

default:
Expand Down Expand Up @@ -116,6 +125,7 @@ function fetch(url, cb) {
if (err) return;
var body;
var type;
var size = req.response.headers["content-length"];
try {
body = JSON.parse(data);
} catch (e) {
Expand All @@ -129,7 +139,8 @@ function fetch(url, cb) {
data = {
text: data,
body: body,
type: type
type: type,
size: size
};
cb(data);
}));
Expand Down