Fix calculating button absolute path length

Fix calculating button absolute path length with non-Latin symbols according issue EdJoPaTo#183
harimambura · Jun 25, 2022 · 30ea891 · 30ea891
1 parent 9514b67
commit 30ea891
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/source/keyboard.ts b/source/keyboard.ts
@@ -78,8 +78,9 @@ function renderRow(templates: readonly ButtonTemplate[], path: string): readonly
 
 function renderCallbackButtonTemplate(template: CallbackButtonTemplate, path: string): InlineKeyboardButton {
 	const absolutePath = combinePath(path, template.relativePath)
-	if (absolutePath.length > 64) {
-		throw new Error(`callback_data only supports 1-64 bytes. With this button (${template.relativePath}) it would get too long (${absolutePath.length}). Full path: ${absolutePath}`)
+	const absolutePathLength = Buffer.byteLength(absolutePath, 'utf-8')
+	if (absolutePathLength > 64) {
+		throw new Error(`callback_data only supports 1-64 bytes. With this button (${template.relativePath}) it would get too long (${absolutePathLength}). Full path: ${absolutePath}`)
 	}
 
 	return {