I solved the challenge with HTTP Method 'DELETE' and app.get. Any problems doing it this way?

Question

I solved the challenge a little differently than Andrew in the video. There are two main differences: 
Instead of using the 'POST'-method I used the 'DELETE'-method. 
In the index.pug-file
JavaScript
extends layout.pug
     block content
          section#content
                   h2 Welcome, #{name}!
                        form(action='/goodbye', method='DELETE')
                             button(type='submit') Goodbye MoonMen

Furthermore, instead of 
JavaScript
app.post('/goodbye', (req, res) => {
   res.clearCookie('username');
   res.redirect('/')
}); 
 
I used 
JavaScript
app.get('/goodbye', (req, res) => {
    res.clearCookie('username', req.body.username);
    res.redirect('/')
});

The alternative ways of solving it seem to work, but I just want to make sure that I have not missed anything by using this method instead. Any and all feedback are much appreciated!

Jamie Gobeille · Accepted Answer

Hello!
I checked and this does seem to work alright in this use case. However, you would not be posting this information to your server, just handling the issue locally in the browser. I can't exactly say this for certain, but I believe your server would still have that cookie stored if you structured your routes this way because you post to the server the username cookie, but never make a post to go and delete it.
Also, you effectively have an extra route that's only purpose is to redirect back to hello. You can type http://localhost:3000/goodbye at any point and get a redirect back to hello. With the post route solution in the video, goodbye isn't an accessible route you can type into the browser to visit.
One other thing is that you don't need to pass in the res.body.username into the res.clearCookie method because the documentation only asks for the name of the cookie, not its value. You can see what I mean by opening the console and going to the application tab, there you will see username under the name and whatever you typed as the username as the value.
```javaScript
app.post("/goodbye", (req, res) => {
//"username" is the cookies name
//res.body.username is the value of the cookie
//This is all that is needed in this case
  res.clearCookie("username");
  res.redirect("hello");
});
```

Welcome to the Treehouse Community

Looking to learn something new?

Kenny Chung

Kenny Chung

I solved the challenge with HTTP Method 'DELETE' and app.get. Any problems doing it this way?

1 Answer

Jamie Gobeille

Jamie Gobeille