Pdf.js: Angular + pdf.js의 IE 11 호환성 활성화

다소 IE11-불완전한 보안 검사 Angular 논리에서 PDF.js 폴리필로 비난을 전가하는 것처럼 들립니다... 좋습니다. 이 격차를 해결하기 위해 간단한 버그로 표시:

1. document.location.origin 는 'origin' 속성이 없으면 새 URL(document.location.href).origin으로 설정해야 합니다.
2. HTMLScriptElement.prototype 는 위와 유사한 논리를 가진 오리진 및 프로토콜 게터를 가져야 합니다.

@yurydelendik 저에게 이것은 MooTools가 부분 폴리필을 적용하고 이에 따라 웹에서 코드가 시작되었기 때문에 몇 가지 Web API의 이름을 변경해야 했던 상황과 유사합니다.

폴리필링은 위험합니다. IMO는 완전한 폴리필로만 수행해야 하며 라이브러리가 아닌 최종 앱에서만 수행해야 합니다. 라이브러리에 어디에서나 사용할 수 없는 특정 API가 필요한 경우 기본 API를 유틸리티 함수로 래핑하고 해당 함수를 사용해야 합니다. 이와 같은 가능한 버그의 전체 클래스를 제거합니다.

요컨대: 라이브러리 코드는 소유하지 않은 개체(예: 기본 브라우저 전역)를 수정해서는 안 됩니다.

폴리필은 위험하다

@mgol 오, 동의합니다. 전체 PDF.js 뷰어 응용 프로그램은 자체 샌드박스(iframe 권장)에 있어야 하지만 사람들은 더 큰 응용 프로그램 내에서 계속 사용합니다. 따라서 그에 따라 대응해야 합니다.

라이브러리 코드는 소유하지 않은 개체(예: 기본 브라우저 전역)를 수정해서는 안 됩니다.

다른 예를 들어보겠습니다. 형식화된 배열이 없으면 Promise PDF.js 라이브러리가 동일하지 않을 것입니다. 그리고 전역 개체를 수정하지 않음으로써 코드를 읽을 수 없게 만들고 대부분의 최신 브라우저에서 성능을 저하시킬 수 있습니다.

다른 예를 들어보겠습니다. 형식화된 배열이 없으면 Promise PDF.js 라이브러리가 동일하지 않을 것입니다. 그리고 전역 개체를 수정하지 않음으로써 코드를 읽을 수 없게 만들고 대부분의 최신 브라우저에서 성능을 저하시킬 수 있습니다.

반드시는 아닙니다. 전역을 가리는 자체 내부 변수 세트를 유지할 수 있습니다. 이것은 모든 경우를 다루지는 않을 수 있지만 적어도 약속은 괜찮을 것입니다. 다음과 같은 것:

var Promise = window.Promise || PROMISE_POLYFILL;

PDF.js 상단에 있습니다. 이 경우 전역을 건드리지 않고 코드에서 변경 없이 Promise 를 계속 사용할 수 있습니다.

이것은 최신 브라우저의 간단한 별칭이므로 성능에 영향을 미치지 않습니다.

모든 경우에 그렇게 쉽지는 않을 수도 있음을 이해합니다(예: 몇 가지 방법만 폴리필해야 하는 경우)

PDF.js 코드가 ES6 모듈을 사용하도록 변환 중입니다. 포장 프로그램이 자동으로 폴리필을 제공하지 않는 한 위의 접근 방식은 atm에서 문제가 될 수 있습니다.

이 문제를 각도 논리 또는 pdf.js 호환성 접근 방식에 대한 논의로 바꾸지 않고 싶습니다. 폴리필을 수정하여 각도를 잘 사용할 수 있도록 하는 것이 좋습니다. PR 환영합니다 :)

다른 생각으로 이 문제는 해결되지 않으므로 종료하겠습니다. Angular+PDF.js+IE11 실제 사용에 대한 확인이 없습니다. 있는 경우 Angular.js를 패치하여 각도 측면에서 쉽게 수정할 수 있습니다.

안녕하세요, 저는 이것이 오래된 주제라는 것을 알고 있지만 위에서 설명한 문제에 직면하고 있습니다.

PDF.js와 함께 Angular 5를 사용하는 프로젝트가 있고 IE11을 지원해야 합니다. 저는 Angular의 초보자이므로 "Angular.js를 패치"하는 방법을 정확히 모르겠습니다. 이 결함을 해결할 수 있는 방법에 대한 지침을 줄 수 있습니까? 미리 감사드립니다.

@elliotstoner 이 문제는 Angular(2+)가 아닌 AngularJS 호환성에 관한 것이며 수동 부트스트랩 대신 ng-app 를 사용하는 경우에만 적용됩니다. Angular 2+는 자동 부트스트래핑도 지원하지 않으므로 이 문제는 적용되지 않습니다.

@mgol Ah ok - 그럼 새로운 이슈를 만들어 보겠습니다. 감사합니다.