🔒 Do not expose application api into (#372)
This commit is contained in:
@@ -23,6 +23,8 @@ export class CompanyApplicationServiceImpl implements TdriveServiceProvider, Ini
|
|||||||
pk.application_id,
|
pk.application_id,
|
||||||
context,
|
context,
|
||||||
);
|
);
|
||||||
|
//do not expose application secret
|
||||||
|
application.api = null;
|
||||||
|
|
||||||
if (!application?.id) {
|
if (!application?.id) {
|
||||||
return null;
|
return null;
|
||||||
@@ -62,6 +64,8 @@ export class CompanyApplicationServiceImpl implements TdriveServiceProvider, Ini
|
|||||||
companyApplication.application_id,
|
companyApplication.application_id,
|
||||||
context,
|
context,
|
||||||
);
|
);
|
||||||
|
//do not expose application secret
|
||||||
|
application.api = null;
|
||||||
if (application)
|
if (application)
|
||||||
applications.push({
|
applications.push({
|
||||||
...companyApplication,
|
...companyApplication,
|
||||||
|
|||||||
@@ -9,7 +9,7 @@ describe("The /users/quota API", () => {
|
|||||||
beforeEach(async () => {
|
beforeEach(async () => {
|
||||||
platform = await init();
|
platform = await init();
|
||||||
currentUser = await UserApi.getInstance(platform);
|
currentUser = await UserApi.getInstance(platform);
|
||||||
}, 30000000);
|
});
|
||||||
|
|
||||||
afterEach(async () => {
|
afterEach(async () => {
|
||||||
await platform.tearDown();
|
await platform.tearDown();
|
||||||
|
|||||||
@@ -14,12 +14,10 @@ import {
|
|||||||
FileTypeUnknownIcon,
|
FileTypeUnknownIcon,
|
||||||
} from '@atoms/icons-colored';
|
} from '@atoms/icons-colored';
|
||||||
import * as Text from '@atoms/text';
|
import * as Text from '@atoms/text';
|
||||||
import { useCompanyApplications } from '@features/applications/hooks/use-company-applications';
|
|
||||||
import { DriveItem } from '@features/drive/types';
|
import { DriveItem } from '@features/drive/types';
|
||||||
import FileUploadAPIClient from '@features/files/api/file-upload-api-client';
|
import FileUploadAPIClient from '@features/files/api/file-upload-api-client';
|
||||||
import { formatDate } from '@features/global/utils/format-date';
|
import { formatDate } from '@features/global/utils/format-date';
|
||||||
import { formatSize } from '@features/global/utils/format-file-size';
|
import { formatSize } from '@features/global/utils/format-file-size';
|
||||||
import useRouterWorkspace from '@features/router/hooks/use-router-workspace';
|
|
||||||
import { useSearchModal } from '@features/search/hooks/use-search';
|
import { useSearchModal } from '@features/search/hooks/use-search';
|
||||||
import { SearchInputState } from '@features/search/state/search-input';
|
import { SearchInputState } from '@features/search/state/search-input';
|
||||||
import { UserType } from '@features/users/types/user';
|
import { UserType } from '@features/users/types/user';
|
||||||
@@ -34,14 +32,8 @@ import useRouterCompany from 'app/features/router/hooks/use-router-company';
|
|||||||
export default (props: { driveItem: DriveItem & { user?: UserType }}) => {
|
export default (props: { driveItem: DriveItem & { user?: UserType }}) => {
|
||||||
const history = useHistory();
|
const history = useHistory();
|
||||||
const input = useRecoilValue(SearchInputState);
|
const input = useRecoilValue(SearchInputState);
|
||||||
const currentWorkspaceId = useRouterWorkspace();
|
|
||||||
const companyApplications = useCompanyApplications();
|
|
||||||
const { user } = useCurrentUser();
|
const { user } = useCurrentUser();
|
||||||
const [_, setParentId] = useRecoilState(DriveCurrentFolderAtom({ initialFolderId: 'user_'+user?.id }));
|
const [_, setParentId] = useRecoilState(DriveCurrentFolderAtom({ initialFolderId: 'user_'+user?.id }));
|
||||||
const tdriveDriveApplicationId =
|
|
||||||
companyApplications.applications.find(application => {
|
|
||||||
return application.identity.code === 'tdrive_drive';
|
|
||||||
})?.id || '';
|
|
||||||
const file = props.driveItem;
|
const file = props.driveItem;
|
||||||
const name = file?.name;
|
const name = file?.name;
|
||||||
const extension = name?.split('.').pop();
|
const extension = name?.split('.').pop();
|
||||||
|
|||||||
Reference in New Issue
Block a user