Using/storing server-side data on the client side

I've a slot booking script which is as follows:

My Current Algorithm

• A calendar is shown to the user

• The user clicks on a date

• I make an ajax call to determine if at least 1 slot is booked on the

clicked date

• I return back a form from the ajax page ("Time To", "Time

From", "Event Name" etc; form along with a hidden input whose value
is "1" if at least 1 slot is booked or "0" if no booking is done for
that day. This hidden input will be used a flag at a later stage)

• The user then enters/selects values in the form and clicks "Book"

• Upon the click of the "Book" button, I check the value of the hidden

input via javascript

• If it is "0", I return true so that the form goes for submission (As there is not even a single slot booked on that day, there is no need to validate if the chosen time slot is booked or not)

• If the value is "1", I run another ajax call to validate whether the time slot selected by the user is free, and if not I return false and display message to user stating the slot is already booked

Now this is where my problem arises. Since the value of the hidden input can be easily manipulated through Firebug before the Book button is clicked, I'm wondering what other way can I set the flag which will be tamper proof by the user.

HTML

JavaScript

```
// executed upon clicking on a date
$("#calendar .day").on("click", function () {
var clicked_date = $(this).find("input").val();
$.ajax({
type: 'post',
url: 'set-flag.php',
data: {
dt: clicked_date
},
success: function (data) {
$("#booking-form").html(data);
}
});
});

//executed upon clicking "Book Date"
$("form#booking-form").on("submit", function ()
{
var flag = $(this).find("input:hidden").val();
if (flag == 0) // form can be submitted
{
return true;
}
else
{
$.ajax({
type: 'post',
url: 'check-slot-availabilit

Right now, you have two separate AJAX calls: one to determine if a day has bookings, and one to determine whether a specific time slot can be booked.

I would suggest one of two things:

• Send a request only once that returns all booking data for the specified date.

• Send only a validation request upon clicking Book; don't bother checking if a date has bookings.

Option 1 is the ideal approach, since it minimises requests. However, if for some reason you don't want a user to know when you're booked, you'll need to go with option 2. Whichever option you go with, you should provide feedback on its success (or failure) after submission, in case somebody else booked it sometime between validation and submission.

If you're adamant about continuing with your approach, then instead of using 0 and 1, consider using a salted hash involving the date. For illustrative purposes, I'm only using one round (since this type of thing doesn't need to be cryptographically secure).

define("BOOKING_DATE_SALT", "asprin111")
$flag = md5($clicked_date . ($db->num_rows() ? "yes" : "no") . BOOKING_DATE_SALT);
...

Of course, you won't want to repeat this code in check-slot-availability.php so it would probably belong better as a function in some shared library.

StackExchange Code Review Q#58256, answer score: 2