Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecated warning: IncomingMessage.connection #764

Closed
developerdavi opened this issue Oct 8, 2022 · 0 comments · Fixed by #765
Closed

Deprecated warning: IncomingMessage.connection #764

developerdavi opened this issue Oct 8, 2022 · 0 comments · Fixed by #765
Labels
back Envolve modificações no backend

Comments

@developerdavi
Copy link
Contributor

Em um ambiente usando TypeScript, me deparei com o seguinte erro:

image

O mesmo trecho de código pode ser encontrado aqui:

function extractAnonymousIpFromRequest(request) {
let ip = request.headers['x-real-ip'] || request.connection.remoteAddress;

É bem simples. O tipo IncomingMessage (@types/node/http.d.ts), qual o NextApiRequest herda propriedades não vai ter mais a propriedade connection em versões futuras do NodeJS. Ao invés disso, você deve usar IncomingMessage.socket – que possui o exato mesmo tipo (imagem abaixo) –, e isso causa um aviso de deprecated a partir da versão 16 do NodeJS e seu respectivo pacote de definição de tipos.

image

Acho que deveríamos usar IncomingMessage.socket, mesmo que o projeto possa ser configurado para rodar usando Node@14. Vou subir um PR mencionando esse issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back Envolve modificações no backend
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant