@sap/cds 7.0.2 → 7.0.3

package/CHANGELOG.md

@@ -4,6 +4,20 @@
 - The format is based on [Keep a Changelog](http://keepachangelog.com/).
 - This project adheres to [Semantic Versioning](http://semver.org/).
 
+## Version 7.0.3 - 2023-07-19
+
+### Fixed
+
+- Compile for lean draft: do not add draft entity for external entities
+- Rollback awaited in REST adapter
+- `service.on('error')` handler invoked only once
+- `SELECT.one.localized`
+- `COPYFILE_DISABLE=1` is now set for building `tar` archives by default
+- Actions of projection target are no longer accessible in linked models
+- Batch execute model-less mass inputs when on `@sap/hana-client`
+- Requests to `/<path>/webapp` return 404 for absolute `@path` specifications
+- `cds compile --to serviceinfo` no longer returns paths w/ Windows `\` path characters
+
 ## Version 7.0.2 - 2023-07-06
 
 ### Fixed

package/lib/compile/for/lean_drafts.js

@@ -106,7 +106,7 @@ module.exports = function cds_compile_for_lean_drafts(csn) {
   }
   for (const name in csn.definitions) {
     const def = csn.definitions[name]
-    if (!_isDraft(def)) continue
+    if (!_isDraft(def) || def['@cds.external']) continue
     def.elements.IsActiveEntity.virtual = true
     def.elements.HasDraftEntity.virtual = true
     def.elements.HasActiveEntity.virtual = true

package/lib/compile/to/srvinfo.js

@@ -62,13 +62,17 @@ module.exports = (model, options={}) => {
 
    // the URL path that is *likely* effective at runtime
   function _url4 (p) {
-    return normalize (p.replace(/^\/+/, '') + '/') //> /foo/bar  ->  foo/bar/
+    p = p.replace(/\\/g, '/') // handle Windows
+         .replace(/^\/+/, '') // strip leading
+         .replace(/\/+$/, '') // strip trailing
+    p += '/' // end with /
+    return p
   }
 
   function _javaPath (service) {
     const d = model.definitions[service.name]
     const path = d && d['@path'] ? d['@path'].replace(/^[^/]/, c => '/'+c) : service.name
-    return join(javaPrefix, path).replace(/\\/g, '/')
+    return join(javaPrefix, path)
   }
 
   function _isNodeProject(root) {

package/lib/linked/models.js

@@ -25,7 +25,8 @@ class LinkedCSN extends any {
         /* else: */         any.prototype
       )
       if (p.key && !d.key && d.kind === 'element') Object.defineProperty (d,'key',{value:undefined})  //> don't propagate .key
-      if (p.params && !d.params && d.kind === 'entity') Object.defineProperty (d,'params',{value:undefined})  //> don't propagate .key
+      if (p.params && !d.params && d.kind === 'entity') Object.defineProperty (d,'params',{value:undefined})  //> don't propagate .params
+      if (p.actions && !d.actions && d.kind === 'entity') Object.defineProperty (d,'actions',{value:undefined})  //> don't propagate .actions
       if (d.elements && d.elements.localized) Object.defineProperty (d,'texts',{value: defs [d.elements.localized.target] })
       try { return Object.setPrototypeOf(d,p) }            //> link d to resolved proto
       catch(e) {                                          //> cyclic proto error

package/lib/ql/SELECT.js

@@ -17,6 +17,7 @@ module.exports = class Query extends Whereable {
       one: $((...x)       => new this({one:true})._select_or_from(...x),{
         columns: (..._)   => new this({one:true}).columns(..._),
         from: (..._)      => new this({one:true}).from(..._),
+        localized: (..._) => new this({one:true,localized:true}).from(..._)
       }),
       from: $((..._)      => new this().from(..._), {
         localized: (..._) => new this({localized:true}).from(..._)

package/lib/srv/protocols/index.js

@@ -83,11 +83,28 @@ class ProtocolAdapter {
           app.use (`/${path}`, before, adapter, after)
         }
 
+        if (!cds.env.features.serve_on_root) {
+          // log warning for changed path if $metadata is accessed
+          let logged = false
+          app.use(`*/${path}/\\$metadata`, (req,res,next) => {
+            if (!logged) {
+              const logger = cds.log('adapters')
+              logger._warn && logger.warn(`With @sap/cds version 7, the service path has changed to '${srv.path}'.
+             If you use SAP Fiori Elements, make sure to adapt the 'dataSources.uri' paths
+             in 'manifest.json' files accordingly. For more information, see the release notes at
+             https://cap.cloud.sap/docs/releases/jun23.`)
+              logged = true
+            }
+            next()
+          })
+        }
+
         prefix = this.protocols[p.kind].path
         prefix = prefix ? (prefix.endsWith('/') ? prefix : prefix + '/') : '/'
         path = prefix + path
       }
 
+      app.use (`${path}/webapp/`, (_,res) => res.sendStatus(404))
       DEBUG?.('app.use(', path, ', ... )')
       app.use (path, before, adapter, after)
       // REVISIT this doesn't handle multiple protocols correctly

package/lib/utils/tar.js

@@ -102,7 +102,7 @@ exports.create = async (dir='.', ...args) => {
       args.push('.')
     }
 
-    c = spawn ('tar', ['c', '-C', dir, ...args])
+    c = spawn ('tar', ['c', '-C', dir, ...args], { env: { COPYFILE_DISABLE: 1 }})
   }
 
   return {__proto__:c, // returning a thenable + fluent ChildProcess...
@@ -169,7 +169,7 @@ exports.extract = (archive, ...args) => ({
   to (...dest) {
     if (typeof dest === 'string') dest = _resolve(...dest)
     const input = typeof archive !== 'string' || archive == '-' ? '-' : _resolve(archive)
-    const x = spawn('tar', ['xf', win(input), '-C', win(dest), ...args], { env: { COPYFILE_DISABLE: 1 }})
+    const x = spawn('tar', ['xf', win(input), '-C', win(dest), ...args])
     if (archive === '-') return x.stdin
     if (Buffer.isBuffer(archive)) archive = require('stream').Readable.from (archive)
     if (typeof archive !== 'string') archive.pipe (x.stdin)

package/libx/_runtime/cds-services/adapter/odata-v4/handlers/error.js

@@ -112,6 +112,7 @@ const getErrorHandler = (crashOnError = true, srv) => {
       if ('_data' in err && !('data' in req)) req.data = err._data
     }
 
+    // REVISIT: invoking service.on('error') handlers needs a cleanup with new protocol adapters!!!
     // invoke srv.on('error', function (err, req) { ... }) here in special situations
     // REVISIT: if for compat reasons, remove once cds^5.1
     if (srv._handlers._error.length) {
@@ -124,7 +125,7 @@ const getErrorHandler = (crashOnError = true, srv) => {
         // > error before req was dispatched
         const creq = new cds.Request({ req, res: req.res, user: req.user || new cds.User.Anonymous() })
         for (const each of srv._handlers._error) each.handler.call(srv, err, creq)
-      } else {
+      } else if (ctx._tx?._done !== 'rolled back') {
         // > error after req was dispatched, e.g., serialization error in okra
         const creq = /* odataReq.req || */ new cds.Request({ req, res: req.res, user: ctx.user, tenant: ctx.tenant })
         for (const each of srv._handlers._error) each.handler.call(srv, err, creq)

package/libx/_runtime/hana/streaming.js

@@ -18,7 +18,8 @@ const _loadStreamExtensionIfNeeded = () => {
 const streamExtension = _loadStreamExtensionIfNeeded()
 
 function hasStreamInsert(insert, model) {
-  if (!model) return true
+  if (!model) return false
+
   const name = insert.into.ref ? insert.into.ref[0] : insert.into
   const into = model.definitions[ensureNoDraftsSuffix(name)]
   if (!into) return false
@@ -38,9 +39,7 @@ function hasStreamInsert(insert, model) {
 }
 
 function hasStreamUpdate(update, model) {
-  if (!model) {
-    return true
-  }
+  if (!model) return false
 
   const entity = model.definitions[ensureNoDraftsSuffix((update.entity.ref && update.entity.ref[0]) || update.entity)]
   if (!entity) return false

package/libx/rest/RestAdapter.js

@@ -194,12 +194,12 @@ const RestAdapter = function (srv) {
 
   // -----------------------------------------------------------------------------------------
   // error handling
-  router.use((err, req, res, next) => {
+  router.use(async (err, req, res, next) => {
     // REVISIT: should not be neccessary!
     // request may fail during processing or during commit -> both caught here
 
     // REVISIT: rollback needed if error occured before commit attempted -> how to distinguish?
-    cds.context?.tx?.rollback(err).catch(() => {}) // REVISIT: silently ?!?
+    await cds.context?.tx?.rollback(err).catch(() => {}) // REVISIT: silently ?!?
 
     next(err)
   })

package/libx/rest/middleware/error.js

@@ -39,13 +39,16 @@ const _log = err => {
 module.exports = (err, req, res, next) => {
   const { _srv: srv } = req
 
+  // REVISIT: invoking service.on('error') handlers needs a cleanup with new protocol adapters!!!
   // invoke srv.on('error', function (err, req) { ... }) here in special situations
   let ctx = cds.context
   if (!ctx) {
     // > error before req was dispatched
     ctx = new cds.Request({ req, res: req.res, user: req.user || new cds.User.Anonymous() })
+    for (const each of srv._handlers._error) each.handler.call(srv, err, ctx)
+  } else if (ctx._tx?._done !== 'rolled back') {
+    for (const each of srv._handlers._error) each.handler.call(srv, err, ctx)
   }
-  for (const each of srv._handlers._error) each.handler.call(srv, err, ctx)
 
   // log the error (4xx -> warn)
   _log(err)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sap/cds",
-  "version": "7.0.2",
+  "version": "7.0.3",
   "description": "SAP Cloud Application Programming Model - CDS for Node.js",
   "homepage": "https://cap.cloud.sap/",
   "keywords": [