emojipacks icon indicating copy to clipboard operation
emojipacks copied to clipboard

emojipacks fails after the most recent slack update

Open 101shipit opened this issue 6 years ago • 18 comments

Slack has update the web UI for the emojis page, and now emojipacks now fails with the following.

Starting import
Got tokens
Logged in
Uh oh! Error: Login error: could not get emoji upload crumb for https://yourteam.slack.com
(node:12714) UnhandledPromiseRejectionWarning: Error: Login error: could not get emoji upload crumb for https://yourteam.slack.com
    at Slack.emoji (/Users/dc/Documents/src/emojipacks/lib/slack.js:162:34)
    at Slack.emoji.next (<anonymous>)
    at onFulfilled (/Users/dc/Documents/src/emojipacks/node_modules/co/index.js:65:19)
    at process._tickCallback (internal/process/next_tick.js:178:7)
(node:12714) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:12714) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

101shipit avatar Aug 28 '18 07:08 101shipit

same error

Ruegen avatar Aug 28 '18 10:08 Ruegen

occur same error

kazuuuutaka avatar Aug 31 '18 05:08 kazuuuutaka

Same error

SatyrTrickster avatar Sep 06 '18 15:09 SatyrTrickster

In my opinion this module needs to be rewritten using puppeteer because Slack has added a modal you must click through.

camwhite avatar Sep 07 '18 22:09 camwhite

@camwhite On it... Slapped this together in a few hours. It's worked well for me so far.

https://github.com/itslenny/slack-emoji-import

itslenny avatar Sep 11 '18 11:09 itslenny

Lol they could only stop bulk uploads for less than a week! Good job @itslenny !!!!

camwhite avatar Sep 11 '18 14:09 camwhite

@camwhite no need for puppeteer, they actually added emoji uploads to the proper api (albeit undocumented), so all you need is a user token. No client crumb or cookie needed so no need to interact with the browser!

jackellenberger avatar Sep 16 '18 22:09 jackellenberger

@lambtron this can be closed with https://github.com/lambtron/emojipacks/pull/155 !

jackellenberger avatar Sep 26 '18 03:09 jackellenberger

@jackellenberger it seems emojipacks still doesn't work - i tried cloning straight from your fork too.

Uh oh! Error: Application Error: unable to find api token on login page
(node:14149) UnhandledPromiseRejectionWarning: Error: Application Error: unable to find api token on login page
    at Slack.login (/Users/michaelmidura/Development/emojipacks/lib/slack.js:156:13)
    at Slack.login.next (<anonymous>)
    at onFulfilled (/Users/michaelmidura/Development/emojipacks/node_modules/co/index.js:65:19)
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:14149) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)

ghost avatar Oct 18 '18 19:10 ghost

@michaelwm I can't reproduce, are you sure you're specifying the correct creds?

I've moved most of my emoji-development time to emojme which does a few more things than emojipacks, but doesn't come with any packs of its own.

jackellenberger avatar Oct 18 '18 21:10 jackellenberger

Hi! It seems that on npm is version without this fix: https://www.npmjs.com/package/emojipacks (last release about year ago)

bronek89 avatar Nov 23 '18 07:11 bronek89

I confirm that cloning the repo solves the issue. Would be great if the latest was published to npm though.

gzurbach avatar Jan 10 '19 18:01 gzurbach

@jackellenberger it seems emojipacks still doesn't work - i tried cloning straight from your fork too.

Uh oh! Error: Application Error: unable to find api token on login page
(node:14149) UnhandledPromiseRejectionWarning: Error: Application Error: unable to find api token on login page
    at Slack.login (/Users/michaelmidura/Development/emojipacks/lib/slack.js:156:13)
    at Slack.login.next (<anonymous>)
    at onFulfilled (/Users/michaelmidura/Development/emojipacks/node_modules/co/index.js:65:19)
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:14149) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)

Also getting this error with correct creds

itsamayo avatar Mar 05 '19 08:03 itsamayo

Hey @AshKetchumza, this is actually a different problem then what was happening before; Slack recently updated their pages to not use a long configuration string that was regex'd into, but as of yesterday they use a nice lil JSON.

That file, /lib/slack.js, will need an update for the login to work again.

jackellenberger avatar Mar 05 '19 18:03 jackellenberger

+1 Still having this error in 2020

xlogix avatar Mar 20 '20 13:03 xlogix

+1 same

davidkhardwick avatar May 06 '20 14:05 davidkhardwick

Anyone find a fix for this?

emeria avatar Jun 02 '21 16:06 emeria

@emeria: no I didn't find a fix. I ended up just using this https://chrome.google.com/webstore/detail/neutral-face-emoji-tools/anchoacphlfbdomdlomnbbfhcmcdmjej

andrewmwilson avatar Jun 02 '21 23:06 andrewmwilson