depau/HackMD
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix parseNoteId order to fix some edge case

Browse source 
that LZString note url could be parsed by base64url note url and thus return wrong note id

Signed-off-by: Max Wu <jackymaxj@gmail.com>
This commit is contained in:
Max Wu 2018-03-10 16:52:24 +08:00
parent 16cb842b94
commit c7657ae81e
1 changed files with 9 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
lib/models/note.js
View file

@ -209,15 +209,6 @@ module.exports = function (sequelize, DataTypes) {
return _callback(err, null)
})
},
parseNoteIdByBase64Url: function (_callback) {
// try to parse note id by base64url
try {
var id = Note.decodeNoteId(noteId)
if (id && Note.checkNoteIdValid(id)) { return callback(null, id) } else { return _callback(null, null) }
} catch (err) {
return _callback(err, null)
}
},
// parse note id by LZString is deprecated, here for compability
parseNoteIdByLZString: function (_callback) {
// try to parse note id by LZString Base64
@ -228,6 +219,15 @@ module.exports = function (sequelize, DataTypes) {
return _callback(err, null)
}
},
parseNoteIdByBase64Url: function (_callback) {
// try to parse note id by base64url
try {
var id = Note.decodeNoteId(noteId)
if (id && Note.checkNoteIdValid(id)) { return callback(null, id) } else { return _callback(null, null) }
} catch (err) {
return _callback(err, null)
}
},
parseNoteIdByShortId: function (_callback) {
// try to parse note id by shortId
try {