useCookie doesn't support options on remove

[JoeDuncko]

What is the current behavior?

useCookie currently doesn't support options on remove. That means that you can't remove any scoped cookies. See https://stackoverflow.com/questions/54861709/cookies-removeabc-not-working-in-reactjs/55593030 and bendotcodes/cookies#16.

Steps to reproduce it and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code and it doesn't have extra dependencies other than react-use. Paste the link to your JSFiddle or CodeSandbox example below:

    const [cookie, setCookie, deleteCookie] = useCookie("cookie");

    setCookie(accessToken, {
            domain: document.domain.match(/[^.]*\.[^.]*$/)?.[0],
        });

    // This doesn't work because it doesn't pass the scope
    deleteCookie();

    // This is what needs to work, but since deleteCookie doesn't support `options`, it won't work
    deleteCookie({
            domain: document.domain.match(/[^.]*\.[^.]*$/)?.[0],
        });

What is the expected behavior?

A little about versions:

• OS: macOS 10.15.6
• Browser (vendor and version): Chrome 85.0.4183.102
• React: 16.13.1
• react-use: 15.3.4
• Did this worked in the previous package version? No

[Boby]

for me I used removeCookie('yourcookiename', { domain: '', path: '/' }); and it clears out but I had to refresh, anyway we can get the cookie domain?

[JoeDuncko]

for me I used removeCookie('yourcookiename', { domain: '', path: '/' }); and it clears out but I had to refresh, anyway we can get the cookie domain?

No, there is no way to get the cookie domain. It's a security function, as the dev should know the domain of the cookie they are manipulating.

[JoeDuncko]

Fixed in react-hookz/web in react-hookz/web#117

[wildfrontend]

so , why not to merge this pull request ?

[JoeDuncko]

so , why not to merge this pull request ?

Unfortunately, react-use is no longer supported, so this PR probably won't ever make it in. See this issue by the last active react-use maintainer.

He's since moved moved on to react-hookz/web.