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.

Sign up for GitHub

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

fix(@yourssu/logging-system): unload event 시 서버로 log 전송 #66

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

fix(@yourssu/logging-system): unload event 시 서버로 log 전송 #66

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
77f9f2c
feat: visibilitychange event로 사용자 이탈 감지
owl1753 Oct 16, 2024
50af6fa
fix: useEffect에 의존성 배열 추가
owl1753 Oct 16, 2024
ad2900e
refactor: async/await 제거
owl1753 Oct 16, 2024
3441ca4
refactor: cleanup 함수 추가
owl1753 Oct 16, 2024
e40b3f4
remove: tcp keepalive 제거
owl1753 Nov 6, 2024
a309429
Merge remote-tracking branch 'origin/main' into fix/send-exit-log
owl1753 Nov 6, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/logging-system/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@
"devDependencies": {
"@types/react-router-dom": "^5.3.3",
"@yourssu/crypto": "workspace:*",
"axios": "^1.6.4",
"axios": "^1.7.7",
"react-router-dom": "^6.21.3"
}
}
25 changes: 24 additions & 1 deletion packages/logging-system/src/contexts/YLSProvider.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
import { createContext } from 'react';
import { createContext, useEffect } from 'react';

import { createAxiosInstance } from '@/apis/createAxiosInstance';
import { LogRequestList, LogResponse } from '@/types/LogType';
import { LogType } from '@/types/LogType';
import { SetLocalStorageClear } from '@/SetLocalStorage';

interface YLSProviderProps {
children: React.ReactNode;
Expand Down Expand Up @@ -32,6 +34,27 @@ export const YLSProvider = ({ children, baseURL }: YLSProviderProps) => {
}
};

useEffect(() => {
const handleVisibilityChange = () => {
if (document.visibilityState === 'visible') return;

const logList: LogType[] = JSON.parse(localStorage.getItem('yls-web') as string) || [];
const data: LogRequestList = {
logRequestList: logList,
};

SetLocalStorageClear();

void axiosInstance.put('/log/list', data);
};

document.addEventListener('visibilitychange', handleVisibilityChange);

return () => {
document.removeEventListener('visibilitychange', handleVisibilityChange);
};
}, []);

return (
<YLSContext.Provider
value={{
Expand Down