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

A bug was found in the CompilerApi.js file under the cubejs-server-core module #9005

Open
tiger-feng opened this issue Nov 29, 2024 · 0 comments
Assignees
Labels
question The issue is a question. Please use Stack Overflow for questions.

Comments

@tiger-feng
Copy link

Describe the bug
In the CompilerApi.js file, there is no requestId in the query parameter of the applyRowLevelSecurity function, which causes this.getCompilers({ requestId: query.requestId }) requestId to always be undefined

To Reproduce
Steps to reproduce the behavior:

  1. Enter the ‘cubejs-server-core/src/core/CompilerApi.js file’
    2.Search for the applyRowLevelSecurity function and see

Expected behavior
requestId should be obtained in context. query.requestId is always undefined.

e.g.
async applyRowLevelSecurity(query, context) {
const compilers = await this.getCompilers({ requestId: context.requestId });
...

Screenshots
image

Version:
[e.g. 1.1.7]

@igorlukanin igorlukanin added the question The issue is a question. Please use Stack Overflow for questions. label Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question The issue is a question. Please use Stack Overflow for questions.
Projects
None yet
Development

No branches or pull requests

3 participants