Skip to content

Commit cb9d3e2

Browse files
chore: log deprecated feature usages
1 parent 1931a73 commit cb9d3e2

File tree

5 files changed

+35
-10
lines changed

5 files changed

+35
-10
lines changed

api/README.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,3 +41,14 @@ npm run start
4141
```sh
4242
npm run dev:pretty
4343
```
44+
45+
## deprecations
46+
47+
- `POST /sconify` is deprecated, websocket API request `SCONIFY_BUILD` should be
48+
used instead.
49+
- `POST /sconify/build` is deprecated, websocket API request `SCONIFY_BUILD`
50+
should be used instead.
51+
- websocket API request `SCONIFY` is deprecated, websocket API request
52+
`SCONIFY_BUILD` should be used instead.
53+
- template `Python` is deprecated, template `Python3.13` should be used instead.
54+
- sconeVersion `v5` is deprecated, sconeVersion `v5.9` should be used instead.

api/src/constants/constants.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,14 +33,14 @@ export const TEMPLATE_CONFIG: Record<
3333
JavaScript: {
3434
// node binary name does not change from one version to another
3535
binary: '/usr/local/bin/node',
36-
// for scone 5.7 this was necessary
36+
// deprecated for scone 5.7 this was necessary
3737
sconeCuratedImage:
3838
'registry.scontain.com:5050/sconecuratedimages/node:14.4.0-alpine3.11',
3939
},
4040
'Python3.13': {
4141
binary: '/usr/local/bin/python3.13',
4242
},
43-
// legacy template name Python used Python 3.8
43+
// deprecated legacy template name Python used Python 3.8
4444
Python: {
4545
binary: '/usr/local/bin/python3.8',
4646
},

api/src/index.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import { requestIdMiddleware } from './utils/requestId.js';
1010
import { errorHandlerMiddleware } from './utils/errors.js';
1111
import { attachWebSocketServer } from './utils/websocket.js';
1212
import {
13-
sconifyBuildHttpHandler,
13+
deprecated_sconifyBuildHttpHandler,
1414
sconifyBuildWsHandler,
1515
} from './sconify/sconifyBuild.handler.js';
1616

@@ -34,7 +34,7 @@ app.use(loggerMiddleware);
3434
// deprecated endpoint, clients should use /sconify/build
3535
app.post('/sconify', deprecated_sconifyHttpHandler);
3636

37-
app.post('/sconify/build', sconifyBuildHttpHandler);
37+
app.post('/sconify/build', deprecated_sconifyBuildHttpHandler);
3838

3939
// Health endpoint
4040
app.get('/health', (req, res) => {

api/src/sconify/deprecated_sconify.handler.ts

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { TEMPLATE_CONFIG, type TemplateName } from '../constants/constants.js';
44
import { ethereumAddressZodSchema } from '../utils/ethereumAddressZodSchema.js';
55
import { deprecated_sconify } from './deprecated_sconify.service.js';
66
import type { Request, Response } from 'express';
7+
import { logger } from '../utils/logger.js';
78

89
const bodySchema = z.object({
910
yourWalletPublicAddress: ethereumAddressZodSchema,
@@ -25,7 +26,7 @@ const bodySchema = z.object({
2526
.default('JavaScript'),
2627
});
2728

28-
async function handleSconifyRequest(requestObj: object) {
29+
async function deprecated_handleSconifyRequest(requestObj: object) {
2930
let yourWalletPublicAddress;
3031
let dockerhubImageToSconify;
3132
let dockerhubPushToken;
@@ -54,18 +55,19 @@ async function handleSconifyRequest(requestObj: object) {
5455
}
5556

5657
export async function deprecated_sconifyWsHandler(message: object) {
58+
logger.warn('deprecated feature hit: ws request SCONIFY');
5759
const { sconifiedImage, appContractAddress } =
58-
await handleSconifyRequest(message);
60+
await deprecated_handleSconifyRequest(message);
5961
return { sconifiedImage, appContractAddress };
6062
}
6163

6264
export async function deprecated_sconifyHttpHandler(
6365
req: Request,
6466
res: Response
6567
) {
66-
const { sconifiedImage, appContractAddress } = await handleSconifyRequest(
67-
req.body || {}
68-
);
68+
logger.warn('deprecated feature hit: POST /sconify');
69+
const { sconifiedImage, appContractAddress } =
70+
await deprecated_handleSconifyRequest(req.body || {});
6971
res.status(200).json({
7072
success: true,
7173
sconifiedImage,

api/src/sconify/sconifyBuild.handler.ts

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import {
88
import { ethereumAddressZodSchema } from '../utils/ethereumAddressZodSchema.js';
99
import { sconify } from './sconifyBuild.service.js';
1010
import type { Request, Response } from 'express';
11+
import { logger } from '../utils/logger.js';
1112

1213
const bodySchema = z.object({
1314
yourWalletPublicAddress: ethereumAddressZodSchema,
@@ -51,6 +52,13 @@ async function handleSconifyRequest(requestObj: object) {
5152
}),
5253
});
5354
}
55+
if (template === 'Python') {
56+
logger.warn('Deprecated feature hit: template === "Python"');
57+
}
58+
if (sconeVersion === 'v5') {
59+
logger.warn('Deprecated feature hit: sconeVersion === "v5"');
60+
}
61+
5462
const { dockerImage, dockerImageDigest, fingerprint, entrypoint } =
5563
await sconify({
5664
dockerImageToSconify: dockerhubImageToSconify,
@@ -85,7 +93,11 @@ export async function sconifyBuildWsHandler(message: object) {
8593
};
8694
}
8795

88-
export async function sconifyBuildHttpHandler(req: Request, res: Response) {
96+
export async function deprecated_sconifyBuildHttpHandler(
97+
req: Request,
98+
res: Response
99+
) {
100+
logger.warn('Deprecated feature hit: POST /sconify/build');
89101
const {
90102
dockerImage,
91103
dockerImageDigest,

0 commit comments

Comments
 (0)