Skip to content

extract licenses does not include all licenses #21309

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
1 of 15 tasks
janwo opened this issue Jul 8, 2021 · 5 comments · Fixed by #21334
Closed
1 of 15 tasks

extract licenses does not include all licenses #21309

janwo opened this issue Jul 8, 2021 · 5 comments · Fixed by #21334

Comments

@janwo
Copy link

janwo commented Jul 8, 2021

🐞 Bug report

Command (mark with an x)

  • new
  • build
  • serve
  • test
  • e2e
  • generate
  • add
  • update
  • lint
  • extract-i18n
  • run
  • config
  • help
  • version
  • doc

Description

Some licenses of project dependencies are missing, e.g. 3rdpartylicenses.txt does not come with licenses of rxjs.

zone.js
MIT
The MIT License

Copyright (c) 2010-2020 Google LLC. https://angular.io/license

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in
all copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
THE SOFTWARE.

🔬 Minimal Reproduction

  1. Create a new project using ng new ("@angular/cli": "^12.1.1")
  2. Run ng build
  3. Check dist/3rdpartylicenses.txt

🔥 Exception or Error

No error is thrown. Some licenses are missing (rxjs).

🌍 Your Environment


    _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/
    

Angular CLI: 12.1.1
Node: 15.13.0 (Unsupported)
Package Manager: npm 7.19.0
OS: darwin arm64

Angular: 12.1.1
... animations, cli, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1201.1
@angular-devkit/build-angular   12.1.1
@angular-devkit/core            12.1.1
@angular-devkit/schematics      12.1.1
@schematics/angular             12.1.1
rxjs                            6.6.7
typescript                      4.3.5

Hope you can help :-)

@alan-agius4
Copy link
Collaborator

Hi @janwo, Unless I am missing something the RxJs JS bundles don't contain any license headers.

Ex: https://unpkg.com/browse/[email protected]/_esm2015/index.js

@janwo
Copy link
Author

janwo commented Jul 8, 2021 via email

@alan-agius4
Copy link
Collaborator

Missed that file, yeah it should consider that. Will try to take a look at this sometime this week. Although likely this is an upstream issue in https://github.com/xz64/license-webpack-plugin.

@alan-agius4 alan-agius4 added the needs: investigation Requires some digging to determine if action is needed label Jul 8, 2021
@alan-agius4
Copy link
Collaborator

Found the root cause xz64/license-webpack-plugin#103

@alan-agius4 alan-agius4 added area: @angular-devkit/build-angular state: blocked on upstream and removed needs: investigation Requires some digging to determine if action is needed labels Jul 9, 2021
@ngbot ngbot bot added this to the needsTriage milestone Jul 9, 2021
@alan-agius4 alan-agius4 added freq1: low Only reported by a handful of users who observe it rarely type: bug/fix freq3: high severity3: broken and removed freq1: low Only reported by a handful of users who observe it rarely labels Jul 9, 2021
@ngbot ngbot bot modified the milestones: needsTriage, Backlog Jul 9, 2021
@alan-agius4 alan-agius4 linked a pull request Jul 13, 2021 that will close this issue
1 task
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Aug 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants