fix: cors options pass by promise

[climba03003]

Resolve: #129

Changes:

• fix cors options pass by promise
• update typings
• update readme

Checklist

• run npm run test and npm run benchmark
• tests and/or benchmarks are included
• documentation is changed or added
• commit message and code follows the Developer's Certification of Origin
  and the Code of conduct

[coveralls]

Pull Request Test Coverage Report for Build 819366967

• 0 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 100.0%

---

Totals
Change from base Build 795457674:	0.0%
Covered Lines:	94
Relevant Lines:	94

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 819366967

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 100.0%

---

Totals
Change from base Build 795457674:	0.0%
Covered Lines:	94
Relevant Lines:	94

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 819383836

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 100.0%

---

Totals
Change from base Build 795457674:	0.0%
Covered Lines:	94
Relevant Lines:	94

---

💛 - Coveralls

[mcollina]

lgtm

[mcollina]

I didn't understand the issue linked, I'm trusting you:(

[zekth]

lgtm

[zekth]

I didn't understand the issue linked, I'm trusting you:(

callback and promise were not handled the same way. Promise was using reply instead of request. See:

fastify-cors/index.js

Line 71 in fe1b50b

optionsResolver(req, (err, options) => {

fastify-cors/index.js

Line 59 in fe1b50b

const ret = optionsResolver(reply)

[climba03003]

I didn't understand the issue linked, I'm trusting you:(

callback and promise were not handled the same way. Promise was using reply instead of request. See:

fastify-cors/index.js

Line 71 in fe1b50b

optionsResolver(req, (err, options) => {

fastify-cors/index.js

Line 59 in fe1b50b

const ret = optionsResolver(reply)

@mcollina zekth explaination is correct for the issue, should we publish as major or patch? Although, it is a fix, we are changing the value passing in the function.

[mcollina]

I'll do a patch - this is a fix.