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

Running npm start throws an exception. Also, logout does not work properly. #27671

Open
1 task done
exemplum opened this issue Oct 24, 2024 · 4 comments
Open
1 task done

Comments

@exemplum
Copy link

Overview of the issue

when you run 'npm start' it displays an error with sass.
setting back following versions fixes the problem
from

"sass": "1.79.2",
"sass-loader": "16.0.1",

to

"sass": "1.72.0",
"sass-loader": "14.1.1",

Also, logout does not work. I cannot catch the error as it quickly redirects it to the landing page.
Previous versions of JHipster would correctly show a page acknowledging a successful logout.

Motivation for or Use Case
Reproduce the error

Both issues are persistent.

Related issues

I could not find anything. I have noticed these issues with JHipster 8.7.0 and 8.7.1

Suggest a Fix

I have already suggested a fix with sass and sass-loader dependency versions. However, I will leave it to you to decide as there may be a better solution.

JHipster Version(s)

8.7.0 and 8.7.1

Yes it is a regression. I have an older version of JHipster working fine.

JHipster configuration

To provide all information we need, you should run jhipster info in the project root folder (or workspaces root for microservices), and
copy/paste the result here.
jhipster info removes sentitive information like rememberKey, jwtSecretKey. Double check if there is any other sensitive info.

Welcome to JHipster v8.7.1

Welcome to the JHipster Information Sub-Generator

[email protected] D:\projects\github\macq
`-- [email protected]
JHipster configuration, a .yo-rc.json file generated in the root folder
.yo-rc.json file
{
  "generator-jhipster": {
    "applicationType": "monolith",
    "authenticationType": "jwt",
    "baseName": "macq",
    "buildTool": "maven",
    "cacheProvider": "ehcache",
    "clientFramework": "react",
    "clientTestFrameworks": [
      "cypress"
    ],
    "clientTheme": "none",
    "creationTimestamp": 1729726001476,
    "cypressAudit": true,
    "cypressCoverage": null,
    "databaseType": "sql",
    "devDatabaseType": "postgresql",
    "devServerPort": 9060,
    "enableHibernateCache": true,
    "enableSwaggerCodegen": false,
    "enableTranslation": true,
    "entities": [],
    "feignClient": null,
    "jhipsterVersion": "8.7.1",
    "languages": [
      "en",
      "sr"
    ],
    "messageBroker": false,
    "microfrontend": null,
    "microfrontends": [],
    "nativeLanguage": "en",
    "packageName": "com.macq",
    "prodDatabaseType": "postgresql",
    "reactive": false,
    "searchEngine": false,
    "serverPort": null,
    "serverSideOptions": [
      "websocket:spring-websocket"
    ],
    "serviceDiscoveryType": false,
    "syncUserWithIdp": null,
    "testFrameworks": [
      "cypress"
    ],
    "websocket": "spring-websocket",
    "withAdminUi": true
  }
}
Environment and Tools

openjdk version "17.0.2" 2022-01-18
OpenJDK Runtime Environment (build 17.0.2+8-86)
OpenJDK 64-Bit Server VM (build 17.0.2+8-86, mixed mode, sharing)

git version 2.45.1.windows.1

node: v20.17.0
npm: 10.9.0

Docker version 20.10.22, build 3a2c30b

JDL for the Entity configuration(s) entityName.json files generated in the .jhipster directory
JDL entity definitions

As alternative you can add a JDL wrapped in below structure

JDL definitions
     JDL content here
  

The information is mandatory for bug reports. This will allow us to use automated tests and genarate the broken sample using jhipster from-issue command.

Browsers and Operating System

Win10 Pro. I use both latest Chrome and latest Brave versions

  • Checking this box is mandatory (this is just to show you read everything)
@mshima
Copy link
Member

mshima commented Oct 24, 2024

Confirmed by #27673

@mshima
Copy link
Member

mshima commented Oct 24, 2024

It happens in development builds only.

@mshima
Copy link
Member

mshima commented Oct 24, 2024

The suggested fix doesn't fix the issue.

@mshima
Copy link
Member

mshima commented Dec 14, 2024

Cypress test is disabled for now.

it<%- clientFrameworkReact ? '.skip' : '' %>('go to home page when successfully logs out', () => {
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants
@mshima @exemplum and others