#3202

Committed 25 Jul 2025 04:52PM UTC coverage: 67.669% (-27.3%) from 94.935%

Build # #3202

Build Type

push

screwdriver

Committed by

web-flow

Commit Message

feat(3363): Update the existing endpoint to get admin for a pipeline from the specified SCM context (#3370)

Run Details

1284 of 2114 branches covered (60.74%)

Branch coverage included in aggregate %.

1 of 11 new or added lines in 1 file covered. (9.09%)

1235 existing lines in 49 files now uncovered.

3417 of 4833 relevant lines covered (70.7%)

50.53 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

26.09

/plugins/secrets/update.js

'use strict';

const boom = require('@hapi/boom');
const joi = require('joi');
const schema = require('screwdriver-data-schema');
const logger = require('screwdriver-logger');
const idSchema = schema.models.secret.base.extract('id');

module.exports = () => ({
    method: 'PUT',
    path: '/secrets/{id}',
    options: {
        description: 'Update a secret',
        notes: 'Update a specific secret',
        tags: ['api', 'secrets'],
        auth: {
            strategies: ['token'],
            scope: ['user', '!guest', 'pipeline']
        },

        handler: async (request, h) => {
            const factory = request.server.app.secretFactory;
            const { credentials } = request.auth;
            const { canAccess } = request.server.plugins.secrets;

            return factory
                .get(request.params.id)
                .then(secret => {
                    if (!secret) {
                        throw boom.notFound('Secret does not exist');
                    }

                    // Make sure that user has permission before updating
                    return canAccess(credentials, secret, 'admin', request.server.app)
                        .then(() => {
                            Object.keys(request.payload).forEach(key => {
                                secret[key] = request.payload[key];
                            });

                            logger.info(
                                `[Audit] user ${credentials.username}:${credentials.scmContext} updates the secret key:${secret.name} for pipelineId:${secret.pipelineId}.`
                            );

                            return secret.update();
                        })
                        .then(() => {
                            const output = secret.toJson();

                            delete output.value;

                            return h.response(output).code(200);
                        });
                })
                .catch(err => {
                    throw err;
                });
        },
        validate: {
            params: joi.object({
                id: idSchema
            }),
            payload: schema.models.secret.update
        }
    }
});

1	'use strict';
2
3	const boom = require('@hapi/boom');	1✔
4	const joi = require('joi');	1✔
5	const schema = require('screwdriver-data-schema');	1✔
6	const logger = require('screwdriver-logger');	1✔
7	const idSchema = schema.models.secret.base.extract('id');	1✔
8
9	module.exports = () => ({	66✔
10	method: 'PUT',
11	path: '/secrets/{id}',
12	options: {
13	description: 'Update a secret',
14	notes: 'Update a specific secret',
15	tags: ['api', 'secrets'],
16	auth: {
17	strategies: ['token'],
18	scope: ['user', '!guest', 'pipeline']
19	},
20
21	handler: async (request, h) => {
UNCOV 22	const factory = request.server.app.secretFactory;	×
UNCOV 23	const { credentials } = request.auth;	×
UNCOV 24	const { canAccess } = request.server.plugins.secrets;	×
25
UNCOV 26	return factory	×
27	.get(request.params.id)
28	.then(secret => {
UNCOV 29	if (!secret) {	×
UNCOV 30	throw boom.notFound('Secret does not exist');	×
31	}
32
33	// Make sure that user has permission before updating
UNCOV 34	return canAccess(credentials, secret, 'admin', request.server.app)	×
35	.then(() => {
UNCOV 36	Object.keys(request.payload).forEach(key => {	×
UNCOV 37	secret[key] = request.payload[key];	×
38	});
39
UNCOV 40	logger.info(	×
41	`[Audit] user ${credentials.username}:${credentials.scmContext} updates the secret key:${secret.name} for pipelineId:${secret.pipelineId}.`
42	);
43
UNCOV 44	return secret.update();	×
45	})
46	.then(() => {
UNCOV 47	const output = secret.toJson();	×
48
UNCOV 49	delete output.value;	×
50
UNCOV 51	return h.response(output).code(200);	×
52	});
53	})
54	.catch(err => {
UNCOV 55	throw err;	×
56	});
57	},
58	validate: {
59	params: joi.object({
60	id: idSchema
61	}),
62	payload: schema.models.secret.update
63	}
64	}
65	});

screwdriver-cd / screwdriver / #3202

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous