Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

when importing video from fakebook thumbnail url seems to long (255) #2343

Closed
albjeremias opened this issue Dec 16, 2019 · 1 comment
Closed
Labels
Type: Bug 🐛 Confirmed bug, at least replicated once by another contributor

Comments

@albjeremias
Copy link

albjeremias commented Dec 16, 2019

What happened?

When i try to upload a file using a URL from facebook I get an error.. and peertube won't import the video. video: "https://www.facebook.com/ptrevolutiontv.live/videos/450177472587941/"

What do you expect to happen instead?
That the video is correctly imported ;) and that peertube would not fail to import the video, just because the import of the thumbnail failed.

Steps to reproduce:

  1. Go to import video from URL

  2. write this URL: https://www.facebook.com/ptrevolutiontv.live/videos/450177472587941/

  3. continue the next steps

  4. Go to Administration -> System -> Logs
    Find the error why your video failed to import.

Additional information

  • PeerTube version or URL: peertube - 2.0.0

  • Link to browser console log if useful:

  "payload": {
    "type": "youtube-dl",
    "videoImportId": 12,
    "thumbnailUrl": "https://scontent-frx5-1.xx.fbcdn.net/v/t15.5256-10/p200x200/75483142_450178619254493_3628944587105501184_n.jpg?_nc_cat=101&_nc_ohc=fj9zbb4XT_8AQkzmTEwzHvPq_gtm5LmrA9sdn2RPyY0Rqq3vX8S5dR2dg&_nc_ht=scontent-frx5-1.xx&oh=7e22daa8d2d723dac65264cc91c45946&oe=5E7566A1",
    "downloadThumbnail": true,
    "downloadPreview": true
  },
  "err": {
    "stack": "SequelizeDatabaseError: value too long for type character varying(255)\n at Query.formatError (/var/www/peertube/node_modules/sequelize/lib/dialects/postgres/query.js:366:16)\n at query.catch.err (/var/www/peertube/node_modules/sequelize/lib/dialects/postgres/query.js:72:18)\n at tryCatcher (/var/www/peertube/node_modules/bluebird/js/release/util.js:16:23)\n at Promise._settlePromiseFromHandler (/var/www/peertube/node_modules/bluebird/js/release/promise.js:547:31)\n at Promise._settlePromise (/var/www/peertube/node_modules/bluebird/js/release/promise.js:604:18)\n at Promise._settlePromise0 (/var/www/peertube/node_modules/bluebird/js/release/promise.js:649:10)\n at Promise._settlePromises (/var/www/peertube/node_modules/bluebird/js/release/promise.js:725:18)\n at _drainQueueStep (/var/www/peertube/node_modules/bluebird/js/release/async.js:93:12)\n at _drainQueue (/var/www/peertube/node_modules/bluebird/js/release/async.js:86:9)\n at Async._drainQueues (/var/www/peertube/node_modules/bluebird/js/release/async.js:102:5)\n at Immediate.Async.drainQueues [as _onImmediate] (/var/www/peertube/node_modules/bluebird/js/release/async.js:15:14)\n at runCallback (timers.js:705:18)\n at tryOnImmediate (timers.js:676:5)\n at processImmediate (timers.js:658:5)",
    "message": "value too long for type character varying(255)",
    "name": "SequelizeDatabaseError",
    "parent": {
      "stack": "error: value too long for type character varying(255)\n at Connection.parseE (/var/www/peertube/node_modules/pg/lib/connection.js:604:11)\n at Connection.parseMessage (/var/www/peertube/node_modules/pg/lib/connection.js:401:19)\n at Socket.<anonymous> (/var/www/peertube/node_modules/pg/lib/connection.js:121:22)\n at Socket.emit (events.js:198:13)\n at addChunk (_stream_readable.js:287:12)\n at readableAddChunk (_stream_readable.js:268:11)\n at Socket.Readable.push (_stream_readable.js:223:10)\n at TCP.onStreamRead [as onread] (internal/stream_base_commons.js:94:17)",
      "message": "value too long for type character varying(255)",
      "name": "error",
      "length": 99,
      "severity": "ERROR",
      "code": "22001",
      "file": "varchar.c",
      "line": "624",
      "routine": "varchar",
      "sql": "INSERT INTO \"thumbnail\" (\"id\",\"filename\",\"height\",\"width\",\"type\",\"fileUrl\",\"automaticallyGenerated\",\"videoId\",\"createdAt\",\"updatedAt\") VALUES (DEFAULT,$1,$2,$3,$4,$5,$6,$7,$8,$9) RETURNING *;",
      "parameters": [
        "9f19d3cf-f0ed-4bb7-b411-69f0e13d8e2c.jpg",
        122,
        223,
        1,
        "https://scontent-frx5-1.xx.fbcdn.net/v/t15.5256-10/p200x200/75483142_450178619254493_3628944587105501184_n.jpg?_nc_cat=101&_nc_ohc=fj9zbb4XT_8AQkzmTEwzHvPq_gtm5LmrA9sdn2RPyY0Rqq3vX8S5dR2dg&_nc_ht=scontent-frx5-1.xx&oh=7e22daa8d2d723dac65264cc91c45946&oe=5E7566A1",
        null,
        44460,
        "2019-12-16 19:09:31.658 +00:00",
        "2019-12-16 19:09:31.658 +00:00"
      ]
    },
    "sql": "INSERT INTO \"thumbnail\" (\"id\",\"filename\",\"height\",\"width\",\"type\",\"fileUrl\",\"automaticallyGenerated\",\"videoId\",\"createdAt\",\"updatedAt\") VALUES (DEFAULT,$1,$2,$3,$4,$5,$6,$7,$8,$9) RETURNING *;"
  }
}
@Chocobozzz
Copy link
Owner

Thanks, should be fixed in 2735a15

@Chocobozzz Chocobozzz added the Type: Bug 🐛 Confirmed bug, at least replicated once by another contributor label Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug 🐛 Confirmed bug, at least replicated once by another contributor
Projects
None yet
Development

No branches or pull requests

2 participants