feat: Remove storage logging and add reset logging (#1342)

PostHog · Aug 5, 2024 · 017d765 · 017d765
1 parent db4d887
commit 017d765
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 13 deletions.
diff --git a/src/posthog-core.ts b/src/posthog-core.ts
@@ -1467,6 +1467,7 @@ export class PostHog {
  * Useful for clearing data when a user logs out.
  */
  reset(reset_device_id?: boolean): void {
+ logger.info('reset')
  if (!this.__loaded) {
  return logger.uninitializedWarning('posthog.reset')
  }

diff --git a/src/storage.ts b/src/storage.ts
@@ -124,7 +124,7 @@ export const cookieStore: PersistentStore = {
  return cookie
  },
 
- set: function (name, value, days, cross_subdomain, is_secure, debug) {
+ set: function (name, value, days, cross_subdomain, is_secure) {
  if (!document) {
  return
  }
@@ -158,10 +158,6 @@ export const cookieStore: PersistentStore = {
  logger.warn('cookieStore warning: large cookie, len=' + new_cookie_val.length)
  }
 
- if (debug) {
- logger.info('cookie set', new_cookie_val)
- }
-
  document.cookie = new_cookie_val
  return new_cookie_val
  } catch (err) {
@@ -232,11 +228,8 @@ export const localStore: PersistentStore = {
  return null
  },
 
- set: function (name, value, _days, _cross_subdomain, _secure, debug) {
+ set: function (name, value) {
  try {
- if (debug) {
- logger.info('localStorage set', name, value)
- }
  window?.localStorage.setItem(name, JSON.stringify(value))
  } catch (err) {
  localStore.error(err)
@@ -383,11 +376,8 @@ export const sessionStore: PersistentStore = {
  return null
  },
 
- set: function (name, value, _days, _cross_subdomain, _secure, debug) {
+ set: function (name, value) {
  try {
- if (debug) {
- logger.info('sessionStorage set', name, value)
- }
  window?.sessionStorage.setItem(name, JSON.stringify(value))
  } catch (err) {
  sessionStore.error(err)