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

Security Officer omas defaults to schema name 'gaian' #5320

Open
planetf1 opened this issue Jun 4, 2021 · 2 comments
Open

Security Officer omas defaults to schema name 'gaian' #5320

planetf1 opened this issue Jun 4, 2021 · 2 comments
Assignees
Labels
pinned Keep open (do not time out) question Not a bug, asking a question

Comments

@planetf1
Copy link
Member

planetf1 commented Jun 4, 2021

When working through PR #5314 I noted that the Security Office OMAS currently has a hardcoded 'gaian' reference where it is defined:

    public static final String DEFAULT_SCHEMA_NAME = "gaian";

This is then used in 'buildContextForTable', and it's not clear there is any support for an alternative schema name - for example:

 public Context buildContextForTable(String userID, String assetId) throws UserNotAuthorizedException, PropertyServerException {
        Context context = new Context();

        context.setTable(getTableName(userID, assetId));
        context.setSchema(Constants.DEFAULT_SCHEMA_NAME);

        return context;
    }

@mandy-chessell
Copy link
Contributor

This is just temporary - will fix it soon.

@github-actions
Copy link

github-actions bot commented Oct 4, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the no-issue-activity Issues automatically marked as stale because they have not had recent activity. label Oct 4, 2021
@planetf1 planetf1 added pinned Keep open (do not time out) and removed no-issue-activity Issues automatically marked as stale because they have not had recent activity. labels Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pinned Keep open (do not time out) question Not a bug, asking a question
Projects
None yet
Development

No branches or pull requests

2 participants