Skip to content

Conversation

@mteodor
Copy link
Member

@mteodor mteodor commented Nov 5, 2021

Signed-off-by: mteodor mirko.teodorovic@gmail.com

When deploying with other applications it may be neccessary to avoid clashing of paths so it may be useful to configure prefix for the whole UI app

so for MF_UI_APP_PREFIX='app' urls like /auth/login become /app/auth/login


import { STRINGS } from 'assets/text/strings';

import { environment } from '../../../../environments/environment';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

environments/environment is ok here

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doesnt work for me

Copy link
Contributor

@manuio manuio Nov 17, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Find a solution please. As you can see we are using absolute path everywhere. For example import { UsersService } from 'app/common/services/users/users.service';

@mteodor mteodor changed the title NOISSUE - Add dynamic prefix conf NOISSUE - Add dynamic prefix conf, and authentication when behind proxy. Nov 16, 2021
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: Manuel Imperiale <manuel.imperiale@gmail.com>
Signed-off-by: Manuel Imperiale <manuel.imperiale@gmail.com>
Signed-off-by: Manuel Imperiale <manuel.imperiale@gmail.com>
Signed-off-by: Manuel Imperiale <manuel.imperiale@gmail.com>
Signed-off-by: Manuel Imperiale <manuel.imperiale@gmail.com>
Signed-off-by: Manuel Imperiale <manuel.imperiale@gmail.com>
Signed-off-by: Manuel Imperiale <manuel.imperiale@gmail.com>
Signed-off-by: Manuel Imperiale <manuel.imperiale@gmail.com>
Signed-off-by: Manuel Imperiale <manuel.imperiale@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Signed-off-by: mteodor <mirko.teodorovic@gmail.com>
Copy link
Contributor

@manuio manuio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@manuio manuio merged commit 45a5a24 into mainflux:master Nov 25, 2021
@mteodor mteodor deleted the path-prefix branch April 20, 2022 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants