#305-fileService #896

a24noabe · 2026-05-11T13:37:05Z

a24noabe commented

2026-05-11 13:37:05 +00:00

FileService

Implemented a service allowing users to upload and download files to a server accessible through http.

Endpoints

Endpoint	Description	Request body
`GET /f/{id}`	Fetch a file	None
`POST /upload/`	Upload a file	A file (<1MB)

Adress

The service is available at /api/files/.
/f/{id} expects the uuid of an uploaded file. It returns the file as an inline file instead of as a downloadable.
/upload/ expects a file, currently only .jpg, .jpeg, and .png files are allowed. The allowed file-types, their corresponding HTTP contenttypes, and their file signatures are added in FileService.cs.
Currently, it doesn't check whether a user is logged in, this functionality will be added once the use of tokens is implemented.
In order to test it, i recommend using an html file:

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Test</title>
</head>
<body>
    <form action="http://localhost/api/files/upload/" method="post" enctype="multipart/form-data">
        <input type="file" name="image" id="image">
        <button type="submit">Upload</button>
    </form>
</body>
</html>

If the file is uploaded successfully you will get a response like:

The name is used at the /f/{id} endpoint, and you can see your image if you go there.

I haven't changed any scripts, so you will have to build the docker image (docker build -t fileservice .) in BoundlessFlowCampus2K\service\fileService\fileService. After you've done this you can run the rebuild script and then the compose file.
Since the endpoints are unsecure, we might have to remove it from the compose file so we don't run it on our server until we have authentication implemented.

# FileService Implemented a service allowing users to upload and download files to a server accessible through http. ## Endpoints | Endpoint | Description | Request body | |---|---|---| | `GET /f/{id}` | Fetch a file | None | | `POST /upload/` | Upload a file | A file (<1MB) | ## Adress The service is available at /api/files/. `/f/{id}` expects the uuid of an uploaded file. It returns the file as an inline file instead of as a downloadable. `/upload/` expects a file, currently only .jpg, .jpeg, and .png files are allowed. The allowed file-types, their corresponding HTTP contenttypes, and their file signatures are added in FileService.cs. Currently, it doesn't check whether a user is logged in, this functionality will be added once the use of tokens is implemented. In order to test it, i recommend using an html file: ```html <!DOCTYPE html> <html lang="en"> <head> <meta charset="UTF-8"> <meta name="viewport" content="width=device-width, initial-scale=1.0"> <title>Test</title> </head> <body> <form action="http://localhost/api/files/upload/" method="post" enctype="multipart/form-data"> <input type="file" name="image" id="image"> <button type="submit">Upload</button> </form> </body> </html> ``` If the file is uploaded successfully you will get a response like: ![image](/attachments/c1c5146d-037f-4d61-afcf-b76384875fa8) The name is used at the `/f/{id}` endpoint, and you can see your image if you go there. I haven't changed any scripts, so you will have to build the docker image (`docker build -t fileservice .`) in `BoundlessFlowCampus2K\service\fileService\fileService`. After you've done this you can run the rebuild script and then the compose file. Since the endpoints are unsecure, we might have to remove it from the compose file so we don't run it on our server until we have authentication implemented.

image.png

3.9 KiB

🚀 1

a24noabe added 8 commits

2026-05-11 13:37:06 +00:00

Start 0fda9ce5df

Ripped straight from tempService.
Nothing done at this point. will get worked on.

More implementing e04add6454

The API endpoints now run, but they don't return images or allow for uploading yet. I've implemented some file validation for the upload to make sure that the files uploaded isn't an exe masquerading as an image file. There is also some debugging code in there but it's commented as such for easy removal. There are improvements to be made

it works 8fd02f2b80

lots of code (unreadable)

did something to fix something e6d6f9d5b0

commented and restructured ed949e82cb

More commenting d84e91818f

Update README.md f93c04d139

directory name change, db connection 9583d9316b

a22erigr added the

needs review

label

2026-05-11 18:16:56 +00:00

a24vinla reviewed

2026-05-12 06:42:45 +00:00

DockerCompose/init_sql/03-files.sql Outdated

					
				@ -0,0 +1,34 @@

				CREATE DATABASE files;   -- Creates and uses the user database for the rest of file

This, or the existing 03-sensors-cron-jobs.sql file, will have to be renamed

~~c24danli referenced this pull request 2026-05-12 08:28:26 +00:00~~

Bulletin Board examples/thoughts #906

a22erigr

2026-05-12 08:36:20 +00:00

added the
Under-review
label
requested review from c22abdal

a22erigr referenced this pull request

2026-05-12 08:48:51 +00:00

Questions for product owner and customer Week 6 #862

c24danli referenced this pull request

2026-05-12 09:24:00 +00:00

db implementation to save pdf in db #909

c22abdal commented

2026-05-12 11:00:52 +00:00

Everything works well and beautifully.
Every changed file is clean, readable, and populated with comments to explain the logic.

I do wonder if the test file should be filled now?

The only thing to improve is probably in the C# files, all the code is clean but to increase the readability, add at least one empty line between the functions/method, also you will be following the code standards.

Once the code standards are followed, this pull request can be merged.

Everything works well and beautifully. Every changed file is clean, readable, and populated with comments to explain the logic. I do wonder if the test file should be filled now? The only thing to improve is probably in the C# files, all the code is clean but to increase the readability, add at least one empty line between the functions/method, also you will be following the code standards. Once the code standards are followed, this pull request can be merged.

c22abdal added

reviewed

and removed

needs review

labels

2026-05-12 11:01:03 +00:00

a22erigr commented

2026-05-12 11:27:31 +00:00

This should not be merged

The developer want to do some changes before merge.

## This should not be merged The developer want to do some changes before merge.

a24noabe added 1 commit

2026-05-12 13:29:12 +00:00

Commenting, standards, and quick fix to method 9e56a8526e

a24noabe added 1 commit

2026-05-12 13:39:28 +00:00

Removed the service from our compose since no auth yet 0d61929861

a22erigr added 1 commit

2026-05-13 06:55:21 +00:00

After fix of merge conflict 6e81f26bcd

a22erigr added 1 commit

2026-05-13 07:02:15 +00:00

Revert "After fix of merge conflict" 09cbbe5973

This reverts commit 6e81f26bcd.

a22erigr added 1 commit

2026-05-13 07:05:39 +00:00

Merge branch 'temp_branch_week_5' into #305-fileService 1acb83a5d7

a22erigr requested review from c24elipe

2026-05-13 07:26:33 +00:00

c24elipe commented

2026-05-13 07:52:09 +00:00

![image](/attachments/8163ba6b-f0ed-4941-9725-6d386b98ed31)

image.png

7.2 KiB

c24elipe commented

2026-05-13 08:01:14 +00:00

![image](/attachments/a2f3dba4-0a23-4a66-b2b3-3713480b5362)

image.png

21 KiB

a24vinla reviewed

2026-05-13 08:08:52 +00:00

service/fileService/fileService/Program.cs

					
				@ -0,0 +14,4 @@

				string DbUser = Environment.GetEnvironmentVariable("DATABASE_USER");

				string DbPassword = Environment.GetEnvironmentVariable("DATABASE_PASSWORD");

				string ConnectionString = $"Host=PostgreSQL;Port=5432;Database=sensors;Username={DbUser};Password={DbPassword}";