Deployed functions req.method not correct for HEAD requests

Version info

firebase-functions: 2.0.3

firebase-tools: 6.1.1

firebase-admin: 6.0.0

node engine: bug appeared in 6 and 8 express: 4.16.3

Test case

Making a head request with curl -I to an express route, while serving locally the req.method is ‘HEAD’. Once deployed however, it is ‘GET’. This is an issue, because I am tracking downloaded files and I don’t want to count head requests which do not download the file.

Steps to reproduce

index.ts:

const echoMethod = (req, res) => {
  const request = {
    method: req.method,
    path: req.path
  }
  return admin.firestore().collection('reqTest').add(request).then(() =>
    res.sendStatus(200);
  ).catch(err => 
    res.sendStatus(500);
  );
}

const testApp = express();
testApp.all('/all', echoMethod);
// have also tried testApp.head(), but same result

const app = express();
app.use('/api', testApp);
export const test = functions.https.onRequest(app);

Then send curl request: curl -I https://myfirebaseapp.com/api/all

Were you able to successfully deploy your functions?

No errors, deploy successful

Expected behavior

req.method should be ‘HEAD’ for head requests. When served locally, this works properly. After deployment, it is recorded as ‘GET’.

Actual behavior

req.method for head request is recorded as ‘GET’.

[thechenky]

Hi @inorganik, this requires some discussion with our partner teams, let me check on that for you. Will post back with any updates.

[thechenky]

Hi @inorganik thanks for the feature request. We are trying not to keep Github issues open if we’re not actively working on them and they are not actual bugs. However, we hear you that this support should exist. I have filed an internal ticket for this request, reference bug 121049520. I’m going to close this out meanwhile.