Remove separate auth params for image endpoints (#212)

These use the same header/cookie auth now as all other api endpoints,
so there is no reason for separate auth param.
This commit is contained in:
Nutomic 2023-11-14 19:26:25 +01:00 committed by GitHub
parent 6f0fab6d7d
commit 8d2364559d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 39 deletions

View file

@ -1375,31 +1375,16 @@ export class LemmyHttp {
/**
* Upload an image to the server.
*/
async uploadImage({
image,
auth,
}: UploadImage): Promise<UploadImageResponse> {
async uploadImage({ image }: UploadImage): Promise<UploadImageResponse> {
const formData = createFormData(image);
// If auth cookie not already set by browser, set it with passed in auth
const headers = {} as any;
if (
!globalThis?.document?.cookie?.includes("auth=") &&
!this.#headers?.Cookie?.includes("auth=")
) {
headers.Cookie = `auth=${auth}`;
}
let url: string | undefined = undefined;
let delete_url: string | undefined = undefined;
const response = await this.#fetchFunction(this.#pictrsUrl, {
method: HttpType.Post,
body: formData as unknown as BodyInit,
headers: {
...this.#headers,
...headers,
},
headers: this.#headers,
});
if (response.status === 413) {
@ -1424,24 +1409,12 @@ export class LemmyHttp {
/**
* Delete a pictrs image
*/
async deleteImage({ token, filename, auth }: DeleteImage): Promise<any> {
// If auth cookie not already set by browser, set it with passed in auth
const headers = {} as any;
if (
!globalThis?.document?.cookie?.includes("auth=") &&
!this.#headers?.Cookie?.includes("auth=")
) {
headers.Cookie = `auth=${auth}`;
}
async deleteImage({ token, filename }: DeleteImage): Promise<any> {
const deleteUrl = `${this.#pictrsUrl}/delete/${token}/${filename}`;
const response = await this.#fetchFunction(deleteUrl, {
method: HttpType.Get,
headers: {
...this.#headers,
...headers,
},
headers: this.#headers,
});
return await response.json();

View file

@ -2,10 +2,6 @@ export const VERSION = "v3";
export interface UploadImage {
image: File | Buffer;
/**
* Optional if cookie with jwt set is already present. Otherwise, auth is required.
*/
auth?: string;
}
export interface UploadImageResponse {
@ -26,8 +22,4 @@ export interface ImageFile {
export interface DeleteImage {
token: string;
filename: string;
/**
* Optional if cookie with jwt set is already present. Otherwise, auth is required.
*/
auth?: string;
}