pnp / cli-microsoft365

Manage Microsoft 365 and SharePoint Framework projects on any platform
https://aka.ms/cli-m365
MIT License
882 stars 312 forks source link

7.5.0 login freeze and CPU go to 100%, but 6.9.0 works fine #5880

Open linchen2chris opened 4 months ago

linchen2chris commented 4 months ago

Priority

(Medium) I'm annoyed but I'll live

Description

hi team, when I run m365 login --authType password --userName chen.lin@xxxx.com --password xxxxxx --debug then it freeze, image but then I switch to m365 6.9.0, everything works.

Steps to reproduce

  1. install m365 7.5.0
  2. run m365 login --authType password -u xxxx -p xxxx --debug

Expected results

it should login success

Actual results

it freeze

Diagnostics

No response

CLI for Microsoft 365 version

3.7.0

nodejs version

18.19.0

Operating system (environment)

Linux

Shell

bash

cli doctor

can't provider doctor, because it need login first

image

Additional Info

No response

waldekmastykarz commented 4 months ago

Thanks for reaching out @linchen2chris and sorry to hear CLI is not working as expected. Could you please try to run v7.0 and see if you still see the error? It would help us a lot to debug the issue, if we could find the first CLI version that shows this behavior. We appreciate your help

martinlingstuyl commented 4 months ago

Just tried 7.5 on Ubuntu 20.04.

martinlingstuyl commented 4 months ago

I'm curious to hear your results when installing 7.0 or 7.4 @linchen2chris

waldekmastykarz commented 4 months ago
  • The cli doctor command throws the same error on my end

cli doctor is not an anonymous command which is why you need to sign in first. That's by design.

linchen2chris commented 3 months ago

I'm curious to hear your results when installing 7.0 or 7.4 @linchen2chris

hi @martinlingstuyl 7.0.0 works

image

but 7.4.0 freeze, same with 7.5.0

image
linchen2chris commented 3 months ago

@martinlingstuyl I also tried 7.3.0 and 7.2.0: 7.2.0 works

image

but 7.3.0 freeze, maybe 7.3.0 is the first version breaks. hope that helps

martinlingstuyl commented 3 months ago

So what kind of Linux environment are you running @linchen2chris ?

linchen2chris commented 3 months ago

So what kind of Linux environment are you running @linchen2chris ?

centos in docker

image image
martinlingstuyl commented 3 months ago

Would you mind sharing your dockerfile. So we can try to reproduce it?

linchen2chris commented 3 months ago

FROM centos

RUN cd /etc/yum.repos.d/ RUN sed -i 's/mirrorlist/#mirrorlist/g' /etc/yum.repos.d/CentOS- RUN sed -i 's|#baseurl=http://mirror.centos.org|baseurl=http://vault.centos.org|g' /etc/yum.repos.d/CentOS- WORKDIR /autotest/ RUN curl -O https://nodejs.org/download/release/v18.19.1/node-v18.19.1-linux-x64.tar.gz \ && tar -xf node-v18.19.1-linux-x64.tar.gz \ && rm node-v18.19.1-linux-x64.tar.gz ENV PATH="/autotest/node-v18.19.1-linux-x64/bin:${PATH}" RUN npm install -g @pnp/cli-microsoft365@6.9.0

waldekmastykarz commented 3 days ago

Hey @martinlingstuyl, did you have a chance to look into this?

martinlingstuyl commented 3 days ago

No unfortunately not, @waldekmastykarz...

martinlingstuyl commented 3 days ago

...but is 7.3 the same version where we updated Ora? We may just have found an additional side effect of the spinner.

martinlingstuyl commented 3 days ago

@Adam-it...

Adam-it commented 3 days ago

...but is 7.3 the same version where we updated Ora? We may just have found an additional side effect of the spinner.

Yes it is. In 7.2 it was 5.4.1 and in 7.3 we updated it to higher version which introduce some problems.

martinlingstuyl commented 3 days ago

@linchen2chris, we'll be removing a faulty package from the CLI in the next beta. We're expecting it may be the reason for your issues. We'll get you up to speed as soon as we've done that so you can try out the new beta. Sorry for the long hold up.