appwrite / appwrite/appwrite

🐛 Bug Report: Geo information headers are not being set for function execution

Open
#9,581 3 comments 0 reactions 0 assignees View on GitHub
bug product / functions
Dominant language
TypeScript
Stars
57.4k
Forks
5.7k
Avg merge
14h 50m
Merged PRs (30d)
276

Description

### 👟 Reproduction steps

When using `createExecution` geo information headers are not being sent to the function.

I suspect this is the problem area, `x-real-ip` is not being set so the geo information section is being skipped over.
https://github.com/appwrite/appwrite/blob/main/app/controllers/api/functions.php#L1848C1-L1859C10

Maybe doing something like what is done in locale would make sense?
https://github.com/appwrite/appwrite/blob/main/app/controllers/api/locale.php#L32

### 👍 Expected behavior

Geo information headers are passed to the function execution.

### 👎 Actual Behavior

Geo information headers are not passed to the function execution.

### 🎲 Appwrite version

Appwrite Cloud

### 💻 Operating system

Windows

### 🧱 Your Environment

sdk-for-web 17.0.1, sdk-for-node 15.0.1

### 👀 Have you spent some time to check if this issue has been raised before?

- [x] I checked and didn't find similar issue

### 🏢 Have you read the Code of Conduct?

- [x] I have read the [Code of Conduct](https://github.com/appwrite/.github/blob/main/CODE_OF_CONDUCT.md)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.