v1.2.0 images fail vulnerability scan

Describe the bug:
All the recently released v1.2.0 images are showing an issue with tzdata out of date.
Image 'cert-manager-acmesolver-amd64:v1.2.0' was last scanned on Thu Mar  4 20:14:45 UTC 2021
The scan results show that 3 ISSUES were found for the image.

Vulnerable Packages Found
=========================

Vulnerability ID   Policy Status   Affected Packages   How to Resolve
DLA-2542-1         Active          tzdata              Upgrade tzdata to >= 2021a-0+deb9u1
DLA-2509-1         Active          tzdata              Upgrade tzdata to >= 2021a-0+deb9u1
DLA-2424-1         Active          tzdata              Upgrade tzdata to >= 2021a-0+deb9u1

-->
Expected behaviour:
No out of date packages.
Anything else we need to know?:
Environment details::

Kubernetes version: v1.18
Cloud-provider/provisioner: IKS
cert-manager version: v1.2.0
Install method: helm

/kind bug

It's worth noting the latest distroless/static image is up to date.
Image 'gcr.io/distroless/static:latest-amd64' was last scanned on Wed Mar  3 23:29:52 UTC 2021
The scan results show that NO ISSUES were found for the image.

OK

Thanks for filing this issue and the PR with the fix.
Is this stopping you from installing/updating to CM 1.2 - is it reported as an error,  or a warning?

There's no severity associated with the issues found.  My personal take is it seems very low priority, but I'm running a local build of the v1.2.0 images with the PR I sent to bypass this issue since we're pushing for 100% clean images at the moment.
Here's the extended output of the vulnerability scanner.
Image '.../quay.io/jetstack/cert-manager-controller:v1.2.0' was last scanned on Wed Mar  3 22:24:27 UTC 2021
The scan results show that 3 ISSUES were found for the image.

Vulnerable Packages Found
=========================

DLA-2542-1

   Policy Status
   Active

   Summary
   This update includes the changes in tzdata 2021a. Notable changes are:

 - South Sudan changed from +03 to +02 on 2021-02-01.

   Vendor Security Notice IDs   Official Notice
   DLA-2542-1                   https://lists.debian.org/debian-lts-announce/2021/02/msg00003.html

   Affected Packages   Policy Status   How to Resolve                        Security Notice
   tzdata              Active          Upgrade tzdata to >= 2021a-0+deb9u1   DLA-2542-1

DLA-2509-1

   Policy Status
   Active

   Summary
   This update includes the changes in tzdata 2020e. Notable changes are:

- - Volgograd switched to Moscow time on 2020-12-27 at 02:00.

   Vendor Security Notice IDs   Official Notice
   DLA-2509-1                   https://lists.debian.org/debian-lts-announce/2020/12/msg00039.html

   Affected Packages   Policy Status   How to Resolve                        Security Notice
   tzdata              Active          Upgrade tzdata to >= 2021a-0+deb9u1   DLA-2509-1

DLA-2424-1

   Policy Status
   Active

   Summary
   tzdata, the time zone and daylight-saving time data, has been updated to the latest version.

  - Revised predictions for Morocco's changes starting in 2023. - Macquarie Island has stayed in sync with Tasmania since 2011. - Casey, Antarctica is at +08 in winter and +11 in summer since 2018. - Palestine ends DST earlier than predicted, on 2020-10-24. - Fiji starts DST later than usual, on 2020-12-20.

   Vendor Security Notice IDs   Official Notice
   DLA-2424-1                   https://lists.debian.org/debian-lts-announce/2020/10/msg00037.html

   Affected Packages   Policy Status   How to Resolve                        Security Notice
   tzdata              Active          Upgrade tzdata to >= 2021a-0+deb9u1   DLA-2424-1

OK


mariadb installs fails and pod enters crashloopbackoff

I tried to install the latest mariadb chart.
I am on minikube v0.14.0 which runs k8s v1.5.1
the chart deploys but I get in a crashloopbackoff:
$ kubectl get pods
NAME                                 READY     STATUS             RESTARTS   AGE
juiced-dog-mariadb-700647841-174wc   0/1       CrashLoopBackOff   33         1h

the describe shows that the probes don't succeed and the pod is restarted:
 1h		2m		127	{kubelet minikube}	spec.containers{juiced-dog-mariadb}	Warning		Unhealthy	Readiness probe failed: mysqladmin: connect to server at 'localhost' failed
error: 'Can't connect to local MySQL server through socket '/opt/bitnami/mariadb/tmp/mysql.sock' (2 "No such file or directory")'
Check that mysqld is running and that the socket: '/opt/bitnami/mariadb/tmp/mysql.sock' exists!

  1h	2m	36	{kubelet minikube}	spec.containers{juiced-dog-mariadb}	Warning	Unhealthy	Liveness probe failed: mysqladmin: connect to server at 'localhost' failed
error: 'Can't connect to local MySQL server through socket '/opt/bitnami/mariadb/tmp/mysql.sock' (2 "No such file or directory")'
Check that mysqld is running and that the socket: '/opt/bitnami/mariadb/tmp/mysql.sock' exists!
`

Can you check if there's anything in the container logs?
kubectl logs -p juiced-dog-mariadb-700647841-174wc

It looks like this issue is the same as https://github.com/kubernetes/charts/issues/228, which on Minikube appears to be caused strangely by a chown line in the entrypoint, which has since been removed (https://github.com/bitnami/bitnami-docker-mariadb/commit/9c858dfd2f55b86896f297c6f68a3d5b285a2a46). https://github.com/kubernetes/charts/pull/427 includes the fix for Minikube at least, or you can use helm install stable/mariadb --set image=bitnami/mariadb:10.1.21-r0 in the meantime.

ok using the new image works.
not sure how changes to the chart will trickle if it is used as a dependency. i.e joomla depends on mariadb. So joomla fails with the 0.5.6 mariadb chart.

thanks for confirming @sebgoa - you're right, once #427 is merged, the dependent charts will also need to be updated to pull in the new version.

@prydonius #427 is merged. I made a PR to update one of the charts #619, then saw your note so I'm going to update that PR to bump them all.

@prydonius Hi, I'm running into the same connection error when running the JasperReports KubeApp chart on Google Container Engine. Its using the MariaDB 10.1.21-r0 image you suggested earlier but I'm still getting the error when I do "kubectl describe "
mysqladmin: connect to server at 'localhost' failed
error: 'Can't connect to local MySQL server through socket '/opt/bitnami/mariadb/tmp/mysql.sock' (2 "No such file or directory")'
Check that mysqld is running and that the socket: '/opt/bitnami/mariadb/tmp/mysql.sock' exists!
whereas the MariaDB server seems to be running fine when I do "kubectl logs  "
2017-04-19 19:48:34<PHONE_NUMBER>19424 [Note] /opt/bitnami/mariadb/sbin/mysqld: ready for connections.
Version: '10.1.21-MariaDB'  socket: '/opt/bitnami/mariadb/tmp/mysql.sock'  port: 3306  Source distribution
Any ideas what I'm doing wrong? I'm not sure what extra details I should provide...

@rbrecheisen are the healthchecks passing for the MariaDB pod? Note that kubectl describe may show warnings about previous healthchecks failing (whilst MariaDB is starting up), even though eventually the healthchecks should pass.

@prydonius Hmmm, good question and thanks for the quick reply. I'll check and let you know asap. As far as I can remember from the loggings the MariaDB does pass the health check. The DB server starts as it should, however, at some point it tries to connect to the server using mysqladmin and it fails with the above-mentioned error. The JasperReports pod does not pass the health check because it fails in a readiness probe:
Get http://<IP_ADDRESS>:8080/jasperserver/: net/http: request canceled (Client.Timeout exceeded while awaiting headers)

Closing due to inactivity, feel free to re-open if this is still an issue.


Add getStore helper + withStore HoC. Change callback + remove Store + add docs + add tests

Work:

[x] Implement getStore helper
[x] Implement withStore HoC
[x] Unify callbacks to 1 callback inside the createStore BREAKING CHANGE.
[x] Implement callback inside useStore and withStore
[x] Remove Store component (now with getStore and callback change it no longer makes sense) BREAKING CHANGE.
[ ] Tests. Some are includes but I want to add more tests
[ ] Docs. Some are included but I to finish it
[x] now occupies 1.2kb 1.08kb 984b, should go down to less than 1kb

Hi, I am Siddharth and I'd like to mention that this is a great library, it is very simple, straightforward and easy to use. Awesome work 👏
I am using it for my university course project and I was looking for something exactly like getStore. I wanted to know if you have an ETA on how long before this PR will be merged?
Thank you.

@siddharthborderwala Thanks a lot! It will take me a while to document and test everything well and maybe some fix / improvement, maybe 1 or 2 days. But since you show interest I have made a prerelease with this in:

0.6.0-canary.1

In theory, everything should work fine, but if you encounter any problem you can comment on it here 🙏 😊
It has some breaking changes that need to be better documented. As we are in an early 0.x stage we try to develop fast to make it smaller, easier and more powerful. It will be more and more stable and from version 1.0 it will be 100% semantic versioning.

Okay, I hear you. Thank you for your time, what I did is use the index.js file inside package directory and converted it to typescript and now I'm using it in my project.

I fix an error on 0.6.0-canary.2

@danielart the PR is finnaly ready for review.


feat: add to nix pkgs

I don't have any experience writing Nix packages. Would you be able to pick this up or point me in the right direction?
I don't imagine it would be crazy, since harper-ls is just a single binary.

Seems that a maintainer has picked it up on the nixpkgs repo

Sweet. Cool if I close this issue then?


GraphicsMagick/ImageMagick: gm couldn't be executed

I'm using this API to get a pdf and this is a snippet of my current set-up:
    formData.append("file", fs.createReadStream("dirname/filepath"))
    const response = await axios.post(
      "https://api.pspdfkit.com/build",
      formData,
      {
        headers: formData.getHeaders({
          Authorization: "Bearer " + LIVE_API_KEY,
        }),
        responseType: "stream",
      }
    )

    await response.data.pipe(fs.createWriteStream(filename))
    const filepath = "dirname/filename"
    const options = {
      density: 100,
      saveFilename: "untitled",
      savePath: "./images",
      format: "png",
      width: 600,
      height: 600,
    }
    setTimeout(() => console.log("times up!"), 1000)
    const convert = fromPath(filepath, options)
    const pageToConvertAsImage = 1

    const result = await convert(pageToConvertAsImage, {
      responseType: "image",
    })
    console.log({ result })

I was getting the pdf from the API mentioned above but I'm getting this error with pdf2image:
/home/myusername/pptToImageApi/node_modules/gm/lib/command.js:249
        cb(new Error('Could not execute GraphicsMagick/ImageMagick: '+cmd+" this most likely means the gm/convert binaries can't be found"));
           ^

Error: Could not execute GraphicsMagick/ImageMagick: gm "convert" "-density" "100x100" "-quality" "75" "-[0]" "-resize" "600x600!" "-compress" "jpeg" "./images/untitled.1.png" this most likely means the gm/convert binaries can't be found
    at ChildProcess.<anonymous> (/home/realsyntexia/nikitimi/pptToImageApi/node_modules/gm/lib/command.js:249:12)
    at ChildProcess.emit (node:events:518:28)
    at ChildProcess._handle.onexit (node:internal/child_process:292:12)
    at onErrorNT (node:internal/child_process:484:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)

It turns out I just need to install GraphicsMagick in my machine
apt-get install graphicsmagick


1.1.7 Cocoapods release

Hi there,
1.1.6 seems to be the latest on Cocoapods, could we get an update made available for 1.1.7 please?

Thanks, I seem to have missed that. 1.1.7 contained a small mistake, 1.1.8 is now available on Cocoapods!


Storing transcripts in blob storage error

Hey everyone,
I am very new to bot framework and using bot framework composer for development.
Right now I am trying to store conversation transcripts in a blob storage. I came across a discussion and saw these steps
In the appsettings.json file; copied
"blobStorage": {
      "connectionString": "*****************",
      "container": "transcripts"
    }

Version
Bot Framework Composer
Version: 2.1.1
OS
Windows
Result:
No transcript is showing up.

@Gupta-Keshav-001378461
Unfortunately that will not work. Blob storage is the only existing storage for transcripts. You could always write your own and base it off the blob transcript store.


Use the original map_id in the plate for map_id in tag 2 index

Closes #
Changes proposed in this pull request:

...

Closed as done.


Adding README.md

What are the relevant tickets?
https://github.com/mitodl/ol-infrastructure/issues/1976
Description (What does it do?)
Added a README detailing dangers of not incrementing version numbers + linking to the build and publish pipeline.
How can this be tested?
N/A

Just came across this PR. Thanks for adding these details @Ardiea . We definitely needed these types of details in our docs/readme.


How to make PR?

I really don't know how to use GitHub. How do I make a pull request?

Hi @cyberchief1984, I would recommend learning the basics of Git version control. Here are some resources:
https://www.youtube.com/watch?v=OqmSzXDrJBk
https://help.github.com/articles/about-pull-requests/
You will first have to fork the repository, then make changes in your fork, and submit those changes in the form of a pull request to this repository. Good luck!


Wrong semantics for accounts not in state trie

Expected Behavior
/account/{address}/balance returns 0 if balance(account) == 0
/tx/contract/create returns "insufficient funds" for accounts with balance lower than needed to deploy contract
/account/{account_pubkey}/nonce returns 0 if nonce(account) == 0
Actual Behavior
Epoch returns "Account not Found" which is semantically incorrect. It should show above behavior as

the balance and nonce of accounts not in the state trie is indeed 0 and could be retuned just like that.
the blockexplorer recieves only a 404 which he can't handle (happens when miner != beneficiary and you click on the miner's address -> error),
the IDE displays a misleading error; debugging would be less irritating, as one might think there is something wrong with the key files which cannot be found for the account one wished to sign the transaction with.

Steps to Reproduce the Problem

Check the balance of an account without balance through API/blockexplorer

or

Try deploying a contract with an account without balance

Hi @nikita-fuchs
We had this discussed in the core-api and api-discussion channels for a couple of times now. Fundamentally question is how shall we treat addresses that are not present in the trie. There are two semantically different approaches:

Treat all addresses as present in the chain, even though those are actually not present in the trie. Then we return balance of 0 even for missing addresses
Make a rapid distinction between present and missing accounts

As you can see - those two approaches provide different types of information. Since the later provides more - we had decided going in that direction. For example in the case where you've put a wrong beneficiery address, the different approaches will result in:

Return 0 for the missing miner address. You don't know the address is not there at all and start thinking something is wrong with the miner reward
Return code 404 and you do know you're checking a missing address and something is wrong

IMO we should addapt the block explorer and the IDE to be ready for parsing responses with code 404 rather than hiding it in epoch and not providing as much valuable information as we can.
Since we are not convinced that we should change the behavior in epoch, I am adding a label wontfix. If you find this description sufficent, you can close the issue. You can also find us in core-api and api-discussion Rocket channels so we can bounce ideas :D

Fair point, Dimitar !
In practice though, there are several scenarios where someone would have to check an address' balance or get its correct nonce, being 0. @jsnewby , would you add it to the SDK, so the user would recieve a a proper result instead of having to evaluate an error message for these two above mentioned cases ?


添加multiDelete的sample，并测试

What would you like to be added:
Why is this needed:

Pls assign to me.

@wang1309


Stateless auth with @AnonymousAllowed endpoint fails after logout.

Description of the bug

Minimal reproducible example

Use https://github.com/vaadin/fusion-crm-tutorial as a basis of test project
Create simple @AnonymousAllowed endpoint (see A below)
Use the endpoint in login-view (see B below)
Add line http.csrf().disable(); to configure(HttpSecurity) method. (edit)
Run the app, login and logout and see the browser's console for the errors

Expected behavior
Calling the @AnonymousAllowed endpoint in login-view should work even after logout.
Actual behavior
401 error is got when calling the endpoint:
Uncaught (in promise) Error: expected "200 OK" response, but got 401
    EndpointError Connect.js:21
    assertResponseIsOk Connect.js:88
    responseHandlerMiddleware Connect.js:208
    chain Connect.js:237
    call Connect.js:245
    _getHello HelloEndpoint.ts:12

Versions:
- Vaadin / Fusion version: 22.0.2 and 23.0.0.alpha1
- Node version:  14.17.0 (global)
- Java version: Amazon Corretto 11
- OS version: Win 11 Pro

I wonder does the Hilla client in browser have an old csrf token after logout, and thus the call to anonymous allowed endpoint fails.
(A)
@Endpoint
@AnonymousAllowed
public class HelloEndpoint {
    public String getHello() {
        return "Hello world";
    }
}

(B)
async connectedCallback() {
    super.connectedCallback();
    this.classList.add('flex', 'flex-col', 'items-center', 'justify-center');
    uiStore.setLoggedIn(false);
    console.log(">>>>"+ await HelloEndpoint.getHello());
  }

Edit: added more complete code example.

Hi, thanks for the feedback.
The jwt cookie is cleared during logout, see the code here.
The described problem here is caused by the HelloEndpoint being called in the connectedCallback, which is called asynchronously while the logout code is running.
If I move the HelloEndpoint call out of the connectedCallback, for example, into a click listener,
<vaadin-button @click=${this.sayHello}>Say Hello</vaadin-button>
async sayHello() {
    console.log(">>>>"+ await HelloEndpoint.getHello());
}

then it works fines.

Thanks @haijian-vaadin you are right that fixes the issue described in the original description. The original description's code example was oversimplified. If I add http.csrf().disable(); line to configure(HttpSecurity http), then  @AnonymousAllowed endpoint breaks after login-logout cycle.

Edited the original bug description to include the above.


Exclude unnecessary dependencies in pinot-clients

As per the issue, removing unnecessary dependencies in pinot-clients

Codecov Report

Merging #11510 (45ed08f) into master (2ab2532) will decrease coverage by 48.60%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #11510       +/-   ##
=============================================
- Coverage     63.03%   14.44%   -48.60%
+ Complexity     1108      201      -907
=============================================
  Files          2320     2320
  Lines        124528   124526        -2
  Branches      19011    19011
=============================================
- Hits          78496    17984    -60512
- Misses        40437   105019    +64582
+ Partials       5595     1523     -4072

Flag
Coverage Δ

integration
?

integration1
?

integration2
?

java-11
14.44% <ø> (-48.57%)
:arrow_down:

java-17
?

java-20
?

temurin
14.44% <ø> (-48.60%)
:arrow_down:

unittests
14.44% <ø> (-48.59%)
:arrow_down:

unittests1
?

unittests2
14.44% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1495 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@cyrilou242, can you please verify and confirm that exclusions are good? Thanks

Hey @abhioncbr I'm not sure this is a good approach in terms of maintainability:

this makes the pom very complex
dependencies in common and core may change. I fear clients exclusion will not be updated accordingly
clients developer may be tempted to use classes of the common and core package that won't work at runtime because some dependencies will have been excluded

I'm not a maintainer of pinot so I don't have a strong opinion but I think it'd be better to first discuss potential solutions in the issue.

Hey @abhioncbr I'm not sure this is a good approach in terms of maintainability:

this makes the pom very complex
dependencies in common and core may change. I fear clients exclusion will not be updated accordingly
clients developer may be tempted to use classes of the common and core package that won't work at runtime because some dependencies will have been excluded

I'm not a maintainer of pinot so I don't have a strong opinion but I think it'd be better to first discuss potential solutions in the issue.

Thanks for the feedback, @cyrilou242, I agree this is not a good solution, and you already outlined the drawbacks.
However, in my understanding, we don't have much of options here, One is to add the exclusion, and the other is either to copy the used code into the java client itself or create a separate submodule and move the dependency code into the new submodule. Please suggest if you have some other options in mind.

@abhioncbr We have to revert his PR because it breaks the JDBC client (unfortunately this PR was included in release 1.0, but the fix was not..) Can you help re-do this PR with a test on the pinot-client? Seems we don't have test on them because the JDBC connector cannot even be instantiated after this PR


Updated X-Client-Version

This modification solves the issue described in https://github.com/tducret/revolut-python/issues/17

Thank you very much @ferensz !


Remove sveltekit:start event

Describe the problem
The only real reason we have this is so that we can do some setup work in our test suite. It's a bit of a weird appendage — you can't really use it to do useful setup work, and it would probably be better to just use root layouts, even if it would add a tiny bit of repeated code to our tests.
Describe the proposed solution
Remove the event
Alternatives considered
Leave the event
Importance
nice to have
Additional Information
No response

It may be that others rely on this to know "ok SvelteKit is setup", maybe KitQL (cc @jycouet), in general all people who came across #4447

I actually didn't know about this event. What I ended up doing in an app (something going back to when it was using Sapper) in e2e tests when I wanted to make sure the app was running and hydrated before proceeding was to have onMount(() => document.body.setAttribute('hydrated', '')) in my root layout and then just wait for that attribute to be present in the tests. That's working fine enough for me, and it's presumably less susceptible to race conditions if you happen to be too later in attaching your event listener.


How to preload my all swiper items?

Hi there? I believe, you can help me!
@jzoom , @jogboms , @iChenLei, @pepsighan
I have a small issue on the flutter swiper plugin. I want to show web view on the each swiper item. Also web view is just contain 1 audio player. When I swap between swiper item, little bit loading. and in this moment, re rendering process, I assume. So, I want to preload all swiper item before show on the screen. Is it possible?

I have the same problem when using Image.network() in my Swiper -- I would love if they could preload somehow!

I solved this by adding allowImplicitScrolling property at PageView.builder in transfomer_page_view.dart.
And use the value true it works.


CAMEL-15573: camel-main: support LambdaRouteBuilder

with the different set of route builders wonder if we just want to make it overloaded method
addRouteBuilder(RouteBuilder)
addRouteBuilder(LambdaRouteBuilder)
withRouteBuilder(RouteBuilder)
withRouteBuilder(LambdaRouteBuilder)

with the different set of route builders wonder if we just want to make it overloaded method
addRouteBuilder(RouteBuilder)
addRouteBuilder(LambdaRouteBuilder)
withRouteBuilder(RouteBuilder)
withRouteBuilder(LambdaRouteBuilder)

I did try but then the call is ambiguous as booth are functional interfaces so in case of a LambdaRouteBuilder, then you need to explicit cast.

Ah okay then we need to use different name.


USE DEDICATED ADMINISTRATIVE ACCOUNTS content is not definitive enough!

3.3: USE DEDICATED ADMINISTRATIVE ACCOUNTS
"Create standard operating procedures around the use of dedicated administrative accounts. Use the recommendations in Azure Security Centre’s "Manage access and permissions" security control to monitor the number of administrative accounts."
This best practice is not definitive enough. i.e. Should I have accounts that just have the "Domain Administrator" role. Or should I have account the has multiple administrator roles. Like an account the has "Exchange Administrator" , "SharePoint Administrator" and "Domain Administrator" roles. Which one is correct?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c3a48c00-8b89-77a1-4a68-62dced213f53
Version Independent ID: d94977b5-050a-21b0-bf9d-d0bca7616e89
Content: Azure Security Control - Identity and Access Control
Content Source: benchmark/azure/security-control-identity-access-control.md
Service: security
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@xedsdead
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Hi, @xedsdead. This article is from the v1 Azure Security Benchmark and is legacy; it is obsolete and will not be updated.  The v2 (and most current) article is https://docs.microsoft.com/en-us/security/benchmark/azure/security-controls-v2-identity-management . Thank you! #please-close


Fix bug causing jasmine runner to fail

Closes #13459. Error was caused by incorrect parsing of jasmine html output. Fixed by looking for different elements.

This should have been addressed by f25c4f11e1d9a3398a99ae0d312decd0b6909413.  If there is still a problem, please file a new bug report and/or a new pull request against the current development sources.


Build Errors

Does the Demo for ProfileService target XAML 2.0 since it has a different version of the Microsoft.NETCore.UWP not even beta.. It throws errors then getting warnings related to Conflicts with versioning on others...

What are you talking about?

My bad

This better?


Updating lnd to ver 0.5 beta rc1

Hi, I've just copied steps from "Install LND" chapter and installed newest version of LND. The system service now fails to start.

admin@raspberrypi:~` $ sudo systemctl status lnd
● lnd.service - LND Lightning Daemon
Loaded: loaded (/etc/systemd/system/lnd.service; enabled; vendor preset: enabled)
Active: activating (auto-restart) (Result: exit-code) since Wed 2018-08-29 11:47:55 CEST; 4s ago
Process: 1160 ExecStart=/usr/local/bin/lnd --externalip=${PUBLICIP} (code=exited, status=1/FAILURE)
Main PID: 1160 (code=exited, status=1/FAILURE)
Aug 29 11:47:55 raspberrypi systemd[1]: lnd.service: Unit entered failed state.
Aug 29 11:47:55 raspberrypi systemd[1]: lnd.service: Failed with result `'exit-code'.

Can you provide a bit more information, please? Log files, config files...

migration to 0.5rc1 is not trivial.
Number of issues are already captured in #190
I had to:

make distinct port numbers in bitcoin.conf
change macaroon location
call lncli with extra parameter (for testnet operation)

I'll document the update once the final version 0.5 is out. Thanks for all the input and y'alls enthusiasm!

i also, tried to upgrade, but when i restart lnd
i receive the following error :
set 15 18:41:41 BitcoinVenetoNode lnd[8890]: Attempting automatic RPC configuration to bitcoind
set 15 18:41:41 BitcoinVenetoNode systemd[1]: lnd.service: Main process exited, code=exited, status=1/FAILURE
set 15 18:41:41 BitcoinVenetoNode systemd[1]: lnd.service: Unit entered failed state.
set 15 18:41:41 BitcoinVenetoNode systemd[1]: lnd.service: Failed with result 'exit-code'.
maybe because of the macaroon file that are not recreated after the restart,
i can only find macaroon file in /home/bitcoin/.lnd and not in /home/bitcoin/.lnd/data/chain/bitcoin/testnet

Guide has been updated. Please let me know if you find any inconsistencies.


[varlib.build] Support different sets of master designs for different glyphs

I am starting to look at how do this. The motivation is that Adobe is planning to deliver a CJK variable font late next year, which will have 64K glyphs. This was developed with our internal tool that supports different sets of master designs for different glyphs, but only a a few hundred of glyph use this feature. I can build this VF now, but only by snapshotting the the source data to make an intermediate master at each transition  between one set of masters and another, in any glyph. This means that instead of having 2 masters for nearly 64K glyphs and 4 to 8 master designs for a small number of other glyphs, the font has 36 masters for 64K glyphs, which is a lot of bloat, and probably larger than the full set of named instances as regular fonts.
There are many issues here, but an  initial API  issue is how decide which glyphs use which masters. I had hoped to be able start with fully populated master designs, and try to figure out for each glyph independently what masters actually affected it, given the data from each of the masters. I would be delighted to hear otherwise, but I have the moment convinced myself that this is not practical for a design with multiple intermediate fonts. My next thought is to specify this externally. I note the designspace file element 'glyph' with the attribute 'mute', which would do this nicely. This is now supported only in the context of  MutatorMath, but I suggest that it be adopted for use in varlib.  Thoughts?

So, what's wrong with this model: masters can have different glyphs.  The base master must have all glyphs.  varLib then checks for each glyph, which masters have it.
Only maters that have GPOS will particilate in GPOS modeling.

While this works on the varLib level, we need to figure out how to get fro Glyphs/UFO to such "sparse" masters which lack GPOS, OS/2, some glyphs, etc.

Let's implement this in varLub' glyf and your CFF2 table and refine from there.

Prepolator is the only workflow

I'm assuming you meant Superpolator

So, what's wrong with this model: masters can have different glyphs. The base master must have all glyphs. varLib then checks for each glyph, which masters have it.
Only maters that have GPOS will particilate in GPOS modeling. Same for other tables (OS/2, ...)

Yes, I think that's what needs to happen.

A few months ago, we added a new layerName attribute to the designspace source element
https://github.com/fonttools/fonttools/pull/1253
One of the primary use cases was indeed to support storing additional sets of masters that only apply to particular glyphs (aka "brace trick" in Glyphs.app jargon).
This is not implemented yet in varLib (https://github.com/fonttools/fonttools/issues/1308).

we need to figure out how to get fro Glyphs/UFO to such "sparse" masters which lack GPOS, OS/2, some glyphs, etc.

The compiler (e.g. fontmake) would need to build this "sparse" masters in the form of incomplete TTF files that only contain the data that varies from the base master, e.g. glyf and hmtx (and probably also maxp, post, etc. the minimum to be able to read them as TTFont); then varLib would check for each glyph for the base master, which other masters contain it, and apply the deltas accordingly.

A few months ago, we added a new layerName attribute to the designspace source element
#1253
One of the primary use cases was indeed to support storing additional sets of masters that only apply to particular glyphs

But the layerName attribute is perpendicular to this issue: a "normal" UFO master should be able to be sparse just the same.

But the layerName attribute is perpendicular to this issue: a "normal" UFO master should be able to be sparse just the same.

Agree.

The compiler (e.g. fontmake) would need to build this "sparse" masters in the form of incomplete TTF files that only contain the data that varies from the base master, e.g. glyf and hmtx (and probably also maxp, post, etc. the minimum to be able to read them as TTFont); then varLib would check for each glyph for the base master, which other masters contain it, and apply the deltas accordingly.

Right.  I can look into the varLib side.  You figure out how the compiler determines this.

Supplying sparse masters needs to be supported, and in many cases will be sufficient, to specify which regions should be used for a given delta tuple. However, I think that there are two cases for which the masters cannot be sparse, and that therefore some meta data is need to indicated which regions can be skipped for some glyphs. The two cases are kern pairs and GPOS contextual statements. Both may involve glyphs that need different sets of regions, and the values in the GPOS tables may be different for all the regions involved, even though some glyphs may use a subset of the regions for their outline data. The source fonts for all these regions will all need to have teh glyps involved. Does anyone see a way around this?

@anthrotype @readroberts I am curious if you have any updates on this, and what the next steps might be? :)

@davelab6  Sparse masters are now well supported on the master branch for variable fonts with TrueType glyph data. I have a branch that supports variable fonts with CFF2 glyph data, which is just awaiting review:
https://github.com/fonttools/fonttools/pull/1475

Fixed via #1591


Login API request example doesn't work - May need to include region

A customer hosted in eu-1 will need to use the following URL instead - which was not obvious from the docs:
https://$CUSTOMER_ID.eu-1.observeinc.com:/v1/login ....

Thanks for raising that! I will adapt the docs.


Adding TLS session key logging capability

Adding the ability to log TLS session keys to a log file for packet
capture decryption purposes.
This adds the following reloadable configuration:
proxy.config.ssl.keylog_file
Since this can work for QUIC as well, this also deprecates:
proxy.config.quic.client.keylog_file

@SolidWallOfCode is going to look at it

[approve ci autest]

I'd be tempted to change the C++ stream logic to use open and writev. The man page claims a single writev will always generate contiguous output even if concurrent. writev also avoids the copy & allocation to put a newline at the end without giving up on the contiguous output with only a read lock.

I'd be tempted to change the C++ stream logic to use open and writev. The man page claims a single writev will always generate contiguous output even if concurrent. writev also avoids the copy & allocation to put a newline at the end without giving up on the contiguous output with only a read lock.

Sounds good. I changed this to use writev.
Thank you for the review and suggestions.

@SolidWallOfCode is going to review it again


Produce a list of entries whose definitions start with number higher than 1

Some entries have multiple definitins like 1) foo bar 2) bar baz etc.
Few of those are incorrectly read, should definition have something that looks like a headword. Lets run a script that outputs entries that look suspicious in that regard. If problematic ones are find (=should not be their own entries), add them to false positives map.

Yeah, seems to produce 700-1000 entries, which is unfortunate. Some seem ok, some seem incorrect. Seems like savotta for a boring day.


ios 8 chrome cached images firing bug

I try to use isotope and imagesloaded (all latest versions). When used on iphone 5, ios 8, chrome browser - initial load is okay, but when I go back and images are cached, isotope fires before imagesloaded.
I guess, it is because images are cached and when browser adds first image on the page -> isotope is fired.
When I add setTimeout after imagesloaded with 2s - images are overlaid on top of each other, but after 2s isotope is fired and everything is okay.
I can reproduce it only on ios 8, chrome browser. desktop or ios safari is okay.
@desandro do you have an iphone and chrome to see this? I can give a link.

I guess, it is the same as this, this, this and others.
Please, resolve this issue, it is been a year it is present :(


The order of label cordinates in different dataset

I noticed the author make some cordinates changes with ICDAR2015 dataset in util.py with "order_points_clockwise" fuction, could you kindly explain the reason for this?
I known the ICDAR2015 dataset has the clockwise label, why did you apply this fuction?
I read the Total-Text dateset paper, could not find any discription about the clockwise label. Do we need to do this kind of changes with the Total-Text dataset?

Were you train Total-Text successfully？Could you please tell me what changes to code need I do?
I want to train ICDAR2019Art dataset, but I find that the coordinates of the polygon bounding boxes of the dataset, which could be 4, 8, 10, 12 polygon vertices.

Were you train Total-Text successfully？Could you please tell me what changes to code need I do?
I want to train ICDAR2019Art dataset, but I find that the coordinates of the polygon bounding boxes of the dataset, which could be 4, 8, 10, 12 polygon vertices.

Sorry, I didnot finish the training of total-text, but there must be lots of changes to be done. Good Luck : ).

I noticed the author make some cordinates changes with ICDAR2015 dataset in util.py with "order_points_clockwise" fuction, could you kindly explain the reason for this?
I known the ICDAR2015 dataset has the clockwise label, why did you apply this fuction?
I read the Total-Text dateset paper, could not find any discription about the clockwise label. Do we need to do this kind of changes with the Total-Text dataset?

Same Question


Change auditing docs page for 1.9 release

Docs PR for the API Audit Logging feature
Doesn't contain much new information, mostly refactoring and making sure new auditing mechanism is viewed as a default.
/cc @sttts @tallclair @CaoShuFeng @ericchiang

This change is

/cc @hzxuzhonghu

/cc @soltysh ^^

This has tech approval from @tengqm. Waiting for tech approval from @tallclair.

@crassirostris 👋 It looks like the preview build is failing due to YAML file inclusion syntax:

12:42:25 PM: jekyll 3.6.0 | Error:  Could not locate the included file 'policy.yaml' in any of ["/opt/build/repo/docs/tasks/debug-application-cluster"]. Ensure it exists in one of those directories and, if it is a symlink, does not point outside your site source.
12:42:25 PM:   Liquid Exception: Could not locate the included file 'policy.yaml' in any of ["/opt/build/repo/docs/tasks/debug-application-cluster"]. Ensure it exists in one of those directories and, if it is a symlink, does not point outside your site source. in docs/tasks/debug-application-cluster/audit.md

@tallclair 👋  Does this look Tech LGTM?

@zacharysarah Oh, thanks a lot for pointing this out! Fixed the link


Connect to API

In this phase, I have completed the following tasks:

Made use of the Bookstore API as your backend.
Used axios with createAsyncThunk to make HTTP requests to the backend.
Fetched my books from the Bookstore API after the component has rendered.
Refactored my add book and remove book features to persist your changes in the server.
The application behaves the same way after the refactoring.

Outcome:

It now works without errors, all my partners also said that it is fine and they have done the same way as I did.
Here is my screenshot showing the actual bookstore without errors.

Ok, it is now fixed and it should work as expected.
This time it does not show any errors when adding books continuously.

Thank you, thank you everybody.
Now in big relief! 🎉😊


Hydration is not working sometimes on some pages

💬 Questions and Help
Hi there.
we are using loadable alongside React18 in our project. on some pages (not all of them) sometimes (not always) react components are being rendered correctly on the server, but the hydration is not working which causes the JS functionality not working on client.
Also, it's good to mention that there is no error in the console and no clue to find the issue. I tried different ways like refactoring the usage of libraries that we are working with like CKEditor and MathJax but none of them solved the issue, so I would like to raise this issue here to see if you have any idea as the hydrarion is being wrapped by loadableReady.

My issue is the hydration, without any warnings or errors creates another duplicate layout instead of hydrating the server html. Both generated HTMLs are identical and react doesn't complaint about differences.
The layout itself is being loaded with loadable. One thing that's different from all examples is that I created a loader for loadable components:
loader.js
function loadComponent(name) {
switch(name) {
case 'ComponentA': {
const Component = loadable(() => /*webpackChunkName: componentA */ import(@namespace/componentA));
  return {
    Component, // render this
    load: Component.load, // used to hookup other things to the framework
    name: 'ComponentA'
  }
}

}
}
App.jsx
import loadComponent from loader.js;
function App({layout}) {
return ({loadComponent(layout).Component});
}
....
entry.js
loadableReady(() => { hydrateRoot(d.gebid('root'), ) });

Yeah, hydration with Suspense can lead to duplicated content instead of hydration failure.
Everything depends on details and without these details, it's hard to help here.

@theKashey thank you for your response. I was able to figure out my issue. It was babel preset-env i made modules set to commonjs this was changing the imports and preventing webpack from splitting. I had stopped the client from processing the loader with babel and this was causing the difference between server and client render.
If you are having issues with loadable components not getting split change preset-env modules from commonjs to the default 'auto'.

Using CommonJS on the client side can also interfere with Webpack tree shaking.
In my corner of IBM, what we do is:

Build client and server separately.
We build client chunks using Babel for JSX, but leave it pure ES6, using ES6 imports. This code is exposed in package.json via 'module'.
We build server side using Babel for JSX + CommonJS transpiling. This code is exposed in package.json via 'main'.
We use rollup to build the same React code twice (for client and server as above).

SSR pass will require react components as CommonJS modules. Meanwhile, Webpack will build dependency tries for chunks using pure ES6 imports and tree shaking works properly, resulting in smaller chunks and proper hydration.

We do 1 and 2. 2 via npm packages (private repo) that are the raw es6 code without processing. Server and client import them and we have 2 webpack configs one for server and one for client.
I don't understand your 3rd and 4th steps? Do you process the packages again with a server side config and commonjs? Do you publish these separately to a repo?
How do you use roll-up? To build the final application? Why roll-up?

You can do (1) and (2) with Webpack alone, for UI microservices. We do (3) and (4) when creating reusable React components - I am sorry for putting them together as 1-4. Again, you don't have to use rollup for 3/4, but it is useful for us to run the same build with different config and copy to esm and cjs dist folders, to be consumed on the client and server, respectively.


fix: [interface] User ID type

Needs to rebase after merging #63
It is too strict that only int can be used for user object ID.

Most of the review will need to wait for rebase.

Rebased.

Rebased after namespace change.


Improve some lexer docstrings

I was reading through the list of lexers and spotted these easy improvements.

Very nice, thanks a lot :) I do appreciate this kind of quality fix; merged!

Thanks for the fast merge. I might do another pass on all of them.


I have seen all messages in #matrix-dev (on another device) but the room gets not unbold.

reported by @krombel:msgs.tk
https://github.com/matrix-org/riot-android-rageshakes/issues/421

Has been fixed: https://github.com/matrix-org/matrix-android-sdk/pull/181


thread 'main' panicked at 'called

Version Information:
swhkd 1.2.1

Distribution Information ( run uname -a )
Linux arch 6.0.12-arch1-1 #1 SMP PREEMPT_DYNAMIC Thu, 08 Dec 2022 11:03:38 +0000 x86_64 GNU/Linux

swhkd version ( swhkd -V )

idk why :(

Can you remove sudo and try again?

Looks like it's crashing while parsing the modifier vector. Can you give us your config file?

Hm thats odd. @EdenQwQ @ajanon any idea what might be going on?

@ Shinyzenith Why i can not use this syntax , and any alter? super + o ; {f, b , t} {firefox, brave, telegram-desktop}
press super + o then leave them and press f to open firefox.

My apologies, I have no experience with modes or our lexer at all. The people I pinged have the most experience and can help you.

This looks a lot like what #171 is trying to fix. I am going to try it and my side and will get back to you.

@vimyumy Would you mind compiling main branch and trying once again? the issue has been fixed.

@ajanon So it looks like that did indeed strip it from the check but now the conditions imposed by the prefixes are no longer respected.

@ajanon So it looks like that did indeed strip it from the check but now the conditions imposed by the prefixes are no longer respected.

Since I don't have a lot of parser experience, @TTWNO Your experience would be beneficial ❤


camel-smpp: CAMEL-8215: update to org.jsmpp / jsmpp v2.1.1

This uses 2.1.1 instead of jumping up to 2.2.0

Applied the patch into camel master branch.

Closing as it is merged in master


Debug JS Remotely

Is this a bug report?
(Yes , 100000000%)

Have you read the Contributing Guidelines?
(Yes)
Environment

react-native -v:

react-native-cli: 2.0.1
react-native: 0.47.1

node -v:

v8.1.3

npm -v:

5.0.3

yarn --version:

0.27.5

Then, specify:

Target Platform: Android

Development Operating System: Windows10

Build tools: react-native run-android

Steps to Reproduce

You cannot easily reproduce error , this is a random cryptic error
Expected Behavior

Executing JS code must not depends on remote debug , So works well when remote debug is off or on
Actual Behavior

For android when remote debug is ON , code works and when OFF not works
Foe more information see:
https://medium.com/@jordan_mohi/cant-find-variable-symbol-in-react-native-782489d86bd7
https://github.com/facebook/react-native/issues/15786
https://github.com/facebook/react-native/issues/15497
Reproducible Demo

(N/A)

I think, it's not a bug, it is a feature.
With Debug enabled, I think for debug support included some additional es6 modules, such as Request, Symbol or some of RegExp features, So, ensure, that You have all dependences included, before start to scripting.
For example,
npm install --save es6-symbol
import 'es6-symbol/implement';

In production mode, you would not need to have unneeded script included.

@iegik I think it is a bug because , I use Symbol in a js file from 6 months ago , I don't change this file from   months ago , suddenly react-native Can’t find variable: Symbol only on android when remote debug is not enabled (see https://github.com/facebook/react-native/issues/15786). If it is not a bug and a feature why iOS has not this feature?
If you think it is a feature , Why we lost BackHandler on android when remote debug is off? (See https://github.com/facebook/react-native/issues/15497)

@sm2017 You can also try to debug with Android Monitor or logcat

Just fyi for people finding this issue via Google, the problem as I understand it is Symbol is not supported in the current version of the JavaScript Core shipped with Android but it is supported natively on iOS which has a newer version of JSC. The reason you don't see this issue when using Remote Debug is because your JavaScript actually runs remotely in Chrome which does support Symbol.
Long story short, use a polyfill for now like es6-symbol.

A possible polyfill solution is mentioned here


4.1.2 Species Search: The characters entered in the search box are used to search for names beginning with those letters, not within the names.

See https://github.com/jdrever/ShropBotSoc/blob/develop/docs/specification.md#412-species-search
Just need to confirm

This is working as expected.


Destroy on dnacenter_sda_fabric_authentication_profile not possible

Prerequisites

[X] Have you tested the operation in the API directly?
[X] Do you have the latest Terraform provider version?
[X] Review the compatibility matrix before opening an issue.

Describe the bug
Destroy on dnacenter_sda_fabric_authentication_profile not possible. As there is as well an error on the API:
{
"status": "failed",
"description": "Cannot unselect the selected authentication profile.",
"taskId": "f83e755e-1559-4fdf-a7bc-620c4bdd8666",
"taskStatusUrl": "/dna/intent/api/v1/task/f83e755e-1559-4fdf-a7bc-620c4bdd8666",
"executionStatusUrl": "/dna/intent/api/v1/dnacaap/management/execution-status/20de3a60-6e71-4982-9786-e3e0a6e549f7",
"executionId": "20de3a60-6e71-4982-9786-e3e0a6e549f7",
"response": {
"failureReason": "Unknown Error: An unexpected condition was encountered. Please try after the system is restored."
}
}
Expected behavior
Delation of authentication profile is not possible. So it should only remove the ressource state.
Screenshots
Error: Failure when executing DeleteDefaultAuthenticationProfileFromSdaFabric
error with operation DeleteDefaultAuthenticationProfileFromSdaFabric
{
"status" : "failed",
"description" : "Cannot unselect the selected authentication profile.",
"taskId" : "a530734b-2254-424b-8fd9-030a2a9736c8",
"taskStatusUrl" : "/dna/intent/api/v1/task/a530734b-2254-424b-8fd9-030a2a9736c8",
"executionStatusUrl" : "/dna/intent/api/v1/dnacaap/management/execution-status/1b568cfd-6191-4f50-897e-416258463dee",
"executionId" : "1b568cfd-6191-4f50-897e-416258463dee"
}
Environment (please complete the following information):

Cisco DNA Center version and patch: <IP_ADDRESS>
Terraform version: 1.3.7
Cisco DNA Center provider version: 1.0.16-beta
OS Version: MacOS 13.1

Additional context
This could be ignored and just remove the state. As there is no option to delete the auth profile from GUI. But fabric site can not be destroyed if the auth profile is still in the state.

I have encountered this issue too. I suppose that better action on resource destruction would be to select default authentication profile in dnac and remove state. In this case terraform effectively rolls back it's own activity to original resource state.
Even better solution would be to implement delete action in DNAC API that will set authentcation profile to default value.

@zapodeanu , in your opinion, what would be the expected behavior of deleting a SDA Fabric Authentication Profile?

@mschedrin If this is still an issue with the API, please open a TAC case

This is not directly an API issue. Current provider implementation does not support destruction of `dnacenter_sda_fabric_authentication_profile´. @fmunozmiranda asked your opinion about what is the better way to destroy the resource using DNAC API. If you don't have any preferences, then I suggest implementing my suggestion in previous comment:

I suppose that better action on resource destruction would be to select default authentication profile in dnac and remove state. In this case terraform effectively rolls back it's own activity to original resource state.

Cleaner implementation would be to implement API endpoint for removing of sda fabric authentication profile. Is something that you can register as feature request for DNAC API @zapodeanu?

I suppose this API call should do the job:

That is correct. The API would be able to delete an auth profile. Are we attempting to remove an auth profile while in use for host onboarding? @mschedrin

Not sure what are use cases for this operation during normal operations, but when running terraform destroy command it is supposed to clean up all the resources/configurations that had been earlier created by terraform, hence we need to remove auth profile configuration and return it to factory defaults.
@fmunozmiranda do you need any more details for implementing this?

Currently I can not verify whether deletion of dnacenter_sda_fabric_authentication_profile resource uses API correctly as creation of this resource does not work either #189.

@zapodeanu, have you heard anything from engineering team yet?

@mschedrin Yes, I did. There is no concept of deleting the default authentication templates from Cisco DNA Center. The current templates - closed, open, low impact or none can not be deleted. We could update a fabric with a PUT to change from using open to none, but we can not delete the template. The delete API does not provide any capabilities today. I do not know if it did in the past, however, at this time will fail as there is not supported fabric auth template operation.

This should be the last operation on fabric site, if the fabric site is in use this operation should be avoided. The templates can not be completed deleted but the delete operation sets it to default 'none'.  If the previous state was none, there will be no change. The error was a defect, which was fixed in <IP_ADDRESS> + releases with CDET:  CSCwd34563.

@DNACENSolutions I cannot open the bug you referenced, it's not public. What has been fixed, has delete method been implemented?


Rich is not working inside docker

Rich does not output colorful text inside docker image.
Docker image inherits from python:3.10.3-slim

Please search closed issues for Docker


Added querying by subgraph id vs deployment id page

Docs entry comparing querying using Subgraph ID vs Deployment ID

Nice clean up @benface. Looking good on my end and ready to merge.
CC @MichaelMacaulay for final review.


Enhancement: show baseline, diff, comparison screenshots in single view

Update the HTML output of report to view baseline, diff and comparison snapshots side by side in a single line for better comparison.

If you're open to it I was also thinking about improving our report, such as updating bootstrap library and showing more information about test suites in general (number of passed or failed tests, .etc). That would be a separate pull request though.
By the way, I don't think that there's any current commit type matching this pull request. It would be nice to have a type of "enhancement"

This is amazing! I will review shortly and merge! I like the idea of adding more information for the report and upgrading the libraries that we are using! keep the PRs coming!

:tada: This PR is included in version 1.20.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


In FUTURE make various QPDFObjectHandle methods const

Making methods const where possible

makes code clearer,
avoids unnecessary copying, and
gives compilers greater scope for optimization.

In order to allow object handle methods to be made const it is necessary to refactor resolving  of objects. In particular, QPDF_Unresolved objects are made responsible for copying, unparsing, etc themselves. Where QPDFObject methods are accessing QPDFValue data members directly rather than through QPDFValue methods, the QPDFObject method is made responsible for handling unresolved objects correctly. At a basic level, this is achieved by modifying the QPDF::resolve method to return a pointer to the resolved object.
In addition to permitting QPDFObjectHandle methods to be made const, this approach to resolving object handles is also more efficient. Accessing unresolved object handles is (relatively) rare: it only applies to indirect objects and only to the first significant access to an object (or even, any object of an object stream). Confining the necessary checks to unresolved object
access only rather than to any object access reduces overhead.

I like the idea and the approach. Would you be willing to rework this without FUTURE now that we are considering the ABI window to be open? It would make the changes easier to review.

I have been thinking about qpdf 12 and would prefer not to break ABI at the moment. In fact, I only reworked the PR yesterday to change it back to FUTURE.

Okay, how would you like me to review this? Are you just looking for my reaction, or do you want a review of specific changes? I had tried a few times in the past to make these const, but I didn't have a solution to the lazy resolve. After the object/value rework, the solution you have put forth seems to be a good one for that problem, and being able to use more const or const& QPDFObjectHandles would be good.

I just sent you a private email to give context to where my head is at the moment.
I see this PR as fundamental for further changes and like to get it it to a mergeable state.


charts: Bump version for Headlamp 0.15.0

@joaquimrocha There's no ghcr.io/kinvolk/headlamp:v0.15.0 released it seems.

@joaquimrocha I think we should merge this now, and get a new chart out.
But I left it un-merged in case you want to do something else first.
(IMHO, we could make other chart updates in the next release and get this one out there.)


Remove mk_sp_lo_plus_one function.

This function is a footgun to usage since lo+1 might not reside on a char boundary.
Previous before #4880 there're only two usages of this function. This pr removes the only remaining usage along-side with it.

Thank you for the PR but I'm going to close this, both because I disagree with the characterization and more generally because it's a change going in the opposite direction of where we need to go.
We really need to make more utility functions available to replace the myriad span derivations happening with bytepos throughout the codebase, including leveraging more of the functions directly provided by the source map (e.g. next_point) that properly handle multibyte characters when we're dealing with span boundaries.
Dropping utility functions is only going to shift such problems out to the corresponding call sites, and I'd much rather have things encapsulated as much as possible


Angular5 update?

The module does not seem to work on the angular 5 version, any update planed? Don't want to recode every dialogs haha
EDITED : My bad, working fine --'

@vanhullc have you tried to delete node_modules and remove the packages related to sweetalert from config and ran npm install --save sweetalert2 @toverux/ngx-sweetalert2 again?
I see that some packages are from 4.x but for me it seems to work currently.

Oh yeah sorry for that working fine now ^^'

I see that some packages are from 4.x but for me it seems to work currently.

Yes, Angular 5 is not officially supported for now (that should come soon), but it should work flawlessly :)


Mesh chooser improvements

Update to 4.7.2 and use ALTER SYSTEM to set mesh_chooser_avoid_redundant_bluetooth to false for this app on both iOS and Android
On iOS this requires wrapping in a Task:
Task { [weak self] in
    do {
        try await ditto.store.execute("ALTER SYSTEM SET mesh_chooser_avoid_redundant_bluetooth = false")
    }
}

As discussed in Slack

This change has been released


Support new Geo types import from WKT

Now that geo types have landed, it would be great to see a pymapd update accommodate them directly.  Currently we are limited to importing geometries as independent columns, or as strings.  The only way to get true points, for example, is to re-export as CSV and then re-import.  This is awkward and inefficient.
Current Behavior:
WKT columns treated as strings
latitude, longitude columns treated as independent numeric columns.
Expected Behavior:
Generally, as in Immerse "Import" and "Copy From" behave.
For adjacent x,y columns, with a table DDL defining Point, have table uploads create Points.
For supported WKT types, have column type set by WKT geometry type (POINT() -> POINT, etc.)

While I agree with the sentiment, I'll have to look into whether our Thrift bindings actually support this. I think it's the case that the geo portion that allows you to load from mapdql is separate from the regular Thrift load_table methods.
So we might not be able to pass the data in without doing some sort subprocess drop into a mapdql shell, which could be tough.

Multiple implementations could support the same functionality from the user perspective.  In terms of architecture and documentation, it would be optimal if the same core code could handle geo transformations, regardless of source.  At the moment, there are slightly-different capabilities depending on the import path (mapdql, thrift, copy from).  Coding geo-Thrift could perhaps be a big task since the given primitives are perhaps not well suited (you might have to re-implement http://www.opengeospatial.org/standards/sfa).  So perhaps cleaner and simpler to implement a post-thrift load_table hook.  This code already exists, since it is used in immerse and 'copy from.'  For example, given separate but adjacent latitude, longitude float columns, it constructs points; given supported WKT, it constructs appropriate geometries.  Perhaps this request could be reformulated as: 'after thrift import, can we apply the same geo-column detection and processing as for import geo'.

Confirmed Randy's load_table_rowwise works as expected.  My notes:
First, the load_table() method given a dataframe won't work as expected by most users. It will create a table with a text column instead of geometry
load_table_rowwise() is currently the only in-memory method which interprets wkt geometry.  However this method requires explicit creation of the table DDL beforehand, and passing the data as Python tuples as not a Pandas dataframe.
Specifically, the rowwise() method requires an iterator returning raw tuples.  Pandas provides a dataframe.itertuples() iterator, which is indeed what we need.  But by default, Pandas returns tuples named "Pandas()" with an index value, neither of which OmniSci will accept.  Passing itertuples() the parameters index=False and name=None gets us raw tuples which work fine!
I'd propose we close this issue and move it to a doc update request for pymapd load_table section.  Probably cleanest in one section called something like "loading geodata"

In support of the geopandas option, here are the steps I currently take (and still need to blog document) for taking a normal pandas dataframe and converting to OmniSci-compatible dataframe:
import geopandas as gpd
import pandas as pd
from shapely.geometry import Point

Create a new coordinates column with tuples of column coordinates (not necessarily WGS84 lat lon):
df['Coordinates'] = list(zip(df.lon, df.lat))

Create binary Shapely points from coordinate tuples (because required for geopandas):
df['Coordinates'] = df['Coordinates'].apply(Point)

Create a new geopandas dataframe (needed temporally in most cases to force projections to 4326):
gdf = geopandas.GeoDataFrame(df, geometry='Coordinates')

Establish input coordinate system from metadata, and set it:
gdf.crs({init: 'EPSG:XXXX'})

Now that set, reproject to  WGS84 lat/lon:
gdf4326 = gdf.to_crs({init: 'EPSG:4326'})

Now that geopandas coordinate column is in correct projection/datum, force back to WKT from binary:
def row2wkt_point(p):
return(p.to_wkt())
df['location'] = df['Coordinates'].apply(row2wkt_point)

Drop extra and or confusing columns:
df.drop({'Coordinates','lat','lon'}, axis=1, inplace=True)

Create a custom DDL for this file:
    conn.execute('CREATE TABLE clim_streamflow (stream_flow float, location GEOMETRY(POINT, 4326), yearint int)')

Finally.... load table rowwise into OmniSci, being sure to avoid the 4 got'chas above (rowwise, without index, unnamed tuples iterator):
    conn.load_table_rowwise("clim_streamflow2060", streamflow2060df.itertuples(index=False, name=None))

So... one of the most commonly-desired open source geodata import pathways requires 9 steps, with 'inside baseball' knowledge that most geospatial or database experts would not have.  I guess I should also note that several of these steps are also computationally time-consuming.  The speed issue is bad enough to be noticeable in raw Python 3.6 even for image tiles of 1k x 1k - and really terrible for LIDAR type applications.
A geopandas import implementation would avoid all the back-and-forth from text to binary and back, and the metadata loss/re-establishment.

Just added a bunch of tests for geospatial row-wise, it's not that complex though it does require creating the table definition upfront:
https://github.com/omnisci/pymapd/blob/master/tests/test_data_no_nulls_cpu.py#L182-L273
I'm hoping to work on the documentation in the next few weeks, making sections for working with various types of data. With a fully worked example in the documentation, hopefully the guessing about what works will start to be minimized.

I'm closing this in lieu of #109, an update to the documentation.


random gathering of use cases

Offline, unsynched changes
I guess that's the first use case that comes to people's mind when they think about oflline-first.
In minutes.io, I show a blue bar at the top of the browser window, telling the user that she/he is working offline, and that there are X unsynched changes.
In Hoodie, devs can subscribe to these events

disconnected
reconnected

and get locally changed objects with hoodie.store.changedObjects()
Signed in, but invalid session
with offline first, you can be signed in and work with the app, but your authentication against the backend might be invalidated.
At minutes.io, I show a yellow exclamation mark sign asking the user that she/he needs to reauthenticate.
In Hoodie, we have an extra events for that

unauthenticated
reauthenticated

With offline-first, there can be a state that changes are stored and visible locally, but have not yet been synchronized to the backend. In that case, you app should defer the sign out and try to push local changes first. But if that doesn't work, the user should be warned about data loos and asked if she/he really wants to sign out

@gr2m why should logging out cause a loss of locally-stored data?  Why can't this data be persisted locally even when the user is logged out?


Add "library" source example

Hello!
Please provide a solution of how to add "library" sources to basic kmp project and use it instead of a prebuild dependency.
I'm trying to fix a few of problems.
While trying a little bit customed "KotlinMP-example" I get spamming from blueFalcon.perepherals.collect { .. }. collect method is called indefinite number of times which I guess is ok and it gives me a set of devices. This set is increased every call by one with as I assume same device somehow copied for unknown for me reason. So spending time I get sets containing huge number of devises like thousands. I suspect it is somehow related to my device(Xiaomi 12 pro). I want to find out how.
So now I find a way to debug and fix the issue at least locally.
Another thing where it might be helpful.
I have another device. I have also tested with it but I have had no luck(no any calls, even permission requests). I conclude it happened because the device is too old. It is Xiaomi MI 8.
I see some limitations in the code. It is minSdk=24 but a firmware is based on Android 10 not 7.
Here I also would like to understand what's what.
Please point out a direction. Thanks!

ok a few questions to deconstruct here.

Solution to add library source. - I need to get some spare time to make the example cleaner and tidy things up, this project has pivoted from being used directly in native, to being used ONLY in KMP. The intention is that you should be able to include blue-falcon in your KMP project, to produce one simple api for all bluetooth actions.
There very well could be a bug with the flow, it was something i put in to make the project more 'reactive' but i should of marked it as unstable, as i havent tested it thoroughly.
How to debug? - Maybe i should write a section in the readme on this, the way i have debugged in the past, checkout the blue-falcon project, publish to maven local, then make sure your KMP project has MavenLocal() in the repositories of the root build gradle/settings. Then when you put a debug point in, it should jump into the source code, you can then put in debugging. I also was thinking of rolling out a logger, which you could plug into, as its quite noisy otherwise.
Android 10 support - Yes this should be working, it must be a permission issue somewhere, this is usally what happens, again if you do the debug, you will be able to break point into where an exception happens upon performing a scan, there could be a custom exception that is being swallowed and ignored.

I hope that gives you some insight, its a tricky thing with bluetooth because you cant easily emulate issues, you usually have to have the device at hand, which makes it more difficult for myself to try and investigate and resolve the issues

I hope that gives you some insight, its a tricky thing with bluetooth because you cant easily emulate issues, you usually
have to have the device at hand, which makes it more difficult for myself to try and investigate and resolve the issues
I see that cumming. Hope I will have good news soon.

Thanks for so fast reply and the insights. Logging would be helpful.

Since then.
I was able to find all the reasons. There are all pretty small. There all related to Android only.
What should I do? PR?
There all pretty small but not all of them are easy to fix. It is sad

Pr would be great, thanks

While trying a little bit customed "KotlinMP-example" I get spamming from blueFalcon.perepherals.collect { .. }. collect method is called indefinite number of times which I guess is ok and it gives me a set of devices. This set is increased every call by one with as I assume same device somehow copied for unknown for me reason.

I ran into the same issue and found a solution by filtering out duplicates from the peripheral set. However I don't know if it was intended by design or if it is really an issue... I just created a PR with my modifications, hope they are useful to someone.

@gitbiagio Hi
Unfortunately due to different issues I haven't been able to get back to it and I won't some time more.
I couldn't find the PR so I've decided to put my question over here.
Was that something related to hash of BluetoothPeripheral there? I found that it troubles when it tries to add new object to the set which supposed to be the same but counts as different

@Tabaqui Hi, you're correct: because BluetoothPeripheral is not a data class, it has no hash function and every instance is different from each other. Building an hash function doesn't resolve the case however, because it ends up adding only the first advertisement seen and never refreshing other data coming with new advertisements (e.g. the rssi value). What I've done is to manually update the set by removing old entries with same uuid before adding new ones.
Look at the PR here #134

Hi guys,
So i have pushed a new change, where i have overridden the equals and hashcode of the bluetooth peripherial, this should stop the excessive reloading when the device has not changed.
I managed to come across an issue with the ios example, because i was using .name as the ID in swiftui, it was rendering the same device multiple times, instead when i use the UUID as the id, it works correctly.

Can i ask, what exactly do you mean by this?
"Please provide a solution of how to add "library" sources to basic kmp project and use it instead of a prebuild dependency."
As i want to help but not sure on the exact question.
Do you want to know how to use KMP libraries in your own KMP library?
Do you want to know how to use KMP libraries in your own KMP project?
Do you want to know how to use KMP libraries in your own native project?

I'm not sure you were saying about me.
If so the case you explained was fine enough. Despite that I have had  some experience in Android I forgot how to act with libraries at all. Also as I said I have been facing some troubles with physical abilities  lately which makes me so slow at some moments.
Well, thanks for helping indeed.

Based on what you have said, it sounds like in general you are struggling with including libraries in your project and this issue is not related to blue falcon but a fundamental in software development.
I will be closing this issue.


⚠ Bofrost.de has degraded performance

In d55bd6b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1900 ms

Resolved: Bofrost.de performance has improved in 85d1395.


[Suggestion] Throw exception if using assertRedirectedTo() without interceptRedirect()

I'm not sure if this is doable, as it seems like interceptRedirect() is a KernelBrowser feature, while assertRedirectTo() is a shared feature. However, I do think it's nice for DX if an error is thrown when asserting a redirection URL if it's followed (or somehow allow this assertion to work when following redirections).

BTW interceptRedirects(), assertRedirected() and assertRedirectedTo() are KernelBrowser and HttpBrowser only features.
This would indeed be helpful. assertRedirected() should get the error thrown as well.

BTW interceptRedirects(), assertRedirected() and assertRedirectedTo() are KernelBrowser and HttpBrowser only features.
This would indeed be helpful. assertRedirected() should get the error thrown as well.

Ah, I think you can merge the Http trait in BrowserKitBrowser directly (I thought its features were reused, but seems like BrowserKitBrowser is the only one using that trait).

Ah, I think you can merge the Http trait in BrowserKitBrowser directly (I thought its features were reused, but seems like BrowserKitBrowser is the only one using that trait).


Empty video in Firefox

the merged recorded video works well in Chrome but empty recorded video in Firefox. If I use just non-merged video, the recorded video displayed. Helps?
`
recordedBlobs = [];
		var merger = new VideoStreamMerger()

		// Add the screen capture. Position it to fill the whole stream (the default)
		merger.addStream(peerStream, {
		  x: 0, // position of the topleft corner
		  y: 0,
		  width: merger.width,
		  height: merger.height,
		  mute: false, // we don't want sound from the screen (if there is any)
		  index: 0
		})

		// Add the webcam stream. Position it on the bottom left and resize it to 100x100.
		merger.addStream(lclStream, {
		  x: merger.width - 100,
		  y: 0,
		  width: 100,
		  height: 100,
		  mute: false,
		  index: null
		})

		// Start the merging. Calling this makes the result available to us
		merger.start()
		combinedStream = merger.result;
		  var options = {mimeType: 'video/webm;codecs=vp9'};
		  if (!MediaRecorder.isTypeSupported(options.mimeType)) {
			console.log(options.mimeType + ' is not Supported');
			options = {mimeType: 'video/webm;codecs=vp8'};
			if (!MediaRecorder.isTypeSupported(options.mimeType)) {
			  console.log(options.mimeType + ' is not Supported');
			  options = {mimeType: 'video/webm'};
			  if (!MediaRecorder.isTypeSupported(options.mimeType)) {
				console.log(options.mimeType + ' is not Supported');
				options = {mimeType: ''};
			  }
			}
		  }
		  try {
			mediaRecorder = new MediaRecorder(combinedStream, options);
			theRecorder = mediaRecorder;
		  } catch (e) {
			console.error('Exception while creating MediaRecorder: ' + e);
			console.log('Exception while creating MediaRecorder: '
			  + e + '. mimeType: ' + options.mimeType);
			return;
		  }
		  console.log('Created MediaRecorder', mediaRecorder, 'with options', options);
		  recordButton.innerHTML = '
Stop Recording
';
		  mediaRecorder.onstop = handleStop;
		  mediaRecorder.ondataavailable = handleDataAvailable;
		  mediaRecorder.start(10); // collect 10ms of data
		  console.log('MediaRecorder started', mediaRecorder);`

Where are the two MediaStreams coming from?
Can you play the combined stream in a video element?
var video = document.createElement('video')
video.src = URL.createObjectURL(merger.result)
document.body.appendChild(video)

Thank you for your prompt reply.
the two MediaStreams are from getUserMedia(); and there's no console errors. However, the combined stream can only be played sometimes and if can be played, the combined video successfully generated. But i am not sure why only sometimes it can be played

That's odd. Do you mind posting your full code and Firefox version?

im afraid that i unable to share the whole code. but I am using SimpleWebRTC, and the firefox version is 55.0.3 (64-bit)

I understand. I'll try and reproduce with what you've put here.

I have simplified the codes and here's the page.

That page didn't work for me. (I think a button is missing, and a way to create peerStream?)
I couldn't reproduce using the example posted here and Firefox 55.0.3.

Click to expand code

    <svg width="80" height="80" viewBox="0 0 250 250" style="fill:#151513; color:#fff; position: absolute; top: 0; border: 0; right: 0;" aria-hidden="true"><path d="M0,0 L115,115 L130,115 L142,142 L250,250 L250,0 Z"></path><path d="M128.3,109.0 C113.8,99.7 119.0,89.6 119.0,89.6 C122.0,82.7 120.5,78.6 120.5,78.6 C119.2,72.0 123.4,76.3 123.4,76.3 C127.3,80.9 125.5,87.3 125.5,87.3 C122.9,97.6 130.6,101.9 134.4,103.2" fill="currentColor" style="transform-origin: 130px 106px;" class="octo-arm"></path><path d="M115.0,115.0 C114.9,115.1 118.7,116.5 119.8,115.4 L133.7,101.6 C136.9,99.2 139.9,98.4 142.2,98.6 C133.8,88.0 127.5,74.4 143.8,58.0 C148.5,53.4 154.0,51.2 159.7,51.0 C160.3,49.4 163.2,43.6 171.4,40.1 C171.4,40.1 176.1,42.5 178.8,56.2 C183.1,58.6 187.2,61.8 190.9,65.4 C194.5,69.0 197.7,73.2 200.1,77.6 C213.8,80.2 216.3,84.9 216.3,84.9 C212.7,93.1 206.9,96.0 205.4,96.6 C205.1,102.4 203.0,107.8 198.3,112.5 C181.9,128.9 168.3,122.5 157.7,114.1 C157.9,116.9 156.7,120.9 152.7,124.9 L141.0,136.5 C139.8,137.7 141.6,141.9 141.8,141.8 Z" fill="currentColor" class="octo-body"></path></svg>

      <video muted autoplay id="one" style="display:inline-block; width: 200px; background:black;"></video>

This makes me think the issue lies with how you are generating these two streams. Any additional details would be helpful.

One thing I noticed is that attempting to use merger.result can sometimes result in a frozen video. Waiting 1 second after merger.start() made the issue disappear.

Open the page in two tabs, you should see your streams, and there's a button on the most bottom right.

And the two streams generated as below.
webrtc.on('localStream', function (stream) {     lclStream = stream; })
webrtc.on('videoAdded', function (video, peer) { 	peerStream = peer.stream; });

Okay, I've managed to reproduce. Seems the generated webm isn't empty, it's corrupted. I'll get back to you on this.

The MIME 'video/webm;codecs=vp9' isn't supported by Firefox.
MediaRecorder.isTypeSupported('video/webm;codecs=vp9') is false
I suggest using mimeType : 'video/mp4' for Firefox, and 'video/webm;codecs=vp9' for Chrome.

it is true that MediaRecorder.isTypeSupported('video/webm;codecs=vp9') is false but MediaRecorder.isTypeSupported('video/webm;codecs=vp8') is true in Firefox. And MediaRecorder.isTypeSupported('video/mp4') is false.
I noticed the Blob is empty therefore I add setTimeout after mediaRecorder.stop() and fixed the issue.
                        mediaRecorder.stop();
   		  setTimeout(function() {
   		  console.log('Recorded Blobs: ', recordedBlobs);
   		  var blob = new Blob(recordedBlobs, {type: 'video/webm'});
   		  var url = window.URL.createObjectURL(blob);
   		  var a = document.createElement('a');
   		  a.style.display = 'none';
   		  a.href = url;
   		  a.download = 'U2DR Record.webm';
   		  document.body.appendChild(a);
   		  a.click();
   		  setTimeout(function() {
   			document.body.removeChild(a);
   			window.URL.revokeObjectURL(url);
   		  }, 100);
   		  }, 1000);


Sync property of Animation Nodes only works with single Animation Nodes. More complex branches are stopped.

Tested versions
Reproducible in Godot 4.3 Dev 5
System information
Ubuntu 22.04
Issue description
The sync property of Animation Nodes are supposed to let both connected branches to go on even at extreme blend amount values. However this only works if such branchs are single animation nodes. This does not work if a branch is more complex, e.g., has Blend2 or other connected nodes.
In the video, we can see that the Blend2 node sync option is enabled. The first connected branch has another blend2 node (Blend2 2). We can see that this first branch is stopped when setting the blend amount to 1 although the sync option is enabled.
This is a problem when trying to synchronize different animation trees, e.g., synchronizing camera and player animation trees.
https://github.com/godotengine/godot/assets/8985409/19d0217a-1d21-4e35-8ba7-dd96d5ad2336
Steps to reproduce
Download the attached project and play with the Blend2 node values to the extremes.
Minimal reproduction project (MRP)
syncanimationnode.zip

This appears to be an old bug that has existed since 3.x.
In 4.x, the behavior of the AnimationNode::_blend_node() function for blend filters and optimization has mostly not changed from 3.x, so there is room for improvement.


In JavaScript, some compile-time errors display as "[object Object]"

Describe the bug
Some types of compile-time errors are reported as "[object Object]" in the Problems panel. It looks like it might be a bundle of multiple error messages that needs to be unpacked.

Hovering on the error squiggle, this pops up:

Repro project: https://makecode.com/_XxTPwyP7cVUW


Mysql storage type problem

Hi Team,
The storage types of Azure database for MySQL are "standard" and "premium" in this article, but they are "Basic" and "General Purpose" on the Azure portal ui. I think it may confuse someone who read this article. Could you please check it? Thanks in advance.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f0f76399-d904-03fe-f033-94c00edda853
Version Independent ID: f79ccda7-bd6e-3204-b8a6-a48840ee6123
Content: Pricing tiers - Azure Database for MySQL
Content Source: articles/mysql/concepts-pricing-tiers.md
Service: mysql
GitHub Login: @jan-eng
Microsoft Alias: janeng

@WenJason Thank you for the feedback.
Yes, this could be confusing.

This issue is being assigned to the content author to evaluate and update the documentation as appropriate.

@WenJason, thank you for reporting his. I have a PR open to change the wording. #please-close


OAStackView arrangedSubviews always return 0

Hi!
OAStackView array arrangedSubviews is always empty. Therefore when I try to insertArrangedSubview: atIndex: the app crashes. I can add view only by using addArrangedSubview.

FWIW, https://github.com/oarrabi/OAStackView/pull/17 will hopefully fix this issue.
As a workaround, you may be able to inspect the stack view's subviews property to decide which index to insert at. No guarantees on that, though -- I've not tried personally.


Remove "babel" field in package.json for react-native support

React-native-packager try to use babel config in package.json or .babelrc and throw "Unknown plugin" error.
This PR will solve this issue and allow using stampit with react-native.
https://github.com/facebook/react-native/issues/4062
react-native TransformError when requiring a node_module that has a package.json with a babel config that specifies uninstalled plugin (stackoverflow)

Wow. Awesome!
We had to put all that code for a reason probably not relevant anymore.
Let me test it back and forth. There might be caveats.
Will publish ASAP.
Cheers

Looks good to me as well. I remember (although bit foggy) that we had to do it like this because of ES2015 preset transpiling those ES modules and babel-preset-es2015-rollup wasn't working that well.
It's bit problematic, that build might actually pass, but don't have exactly tools to automatically verify that those builds are really working properly.

First of all, thank you very much @Piroro-hs for that cool deep techy contribution.
I checked the final resulting bundle.

The new bundle now has this code:

var _typeof = typeof Symbol === "function" && typeof Symbol.iterator === "symbol" ? function (obj) { return typeof obj; } : function (obj) { return obj && typeof Symbol === "function" && obj.constructor === Symbol && obj !== Symbol.prototype ? "symbol" : typeof obj; };

which replaces the typeof used in the isObject() and isPlainObject(). It's done to support symbols I assume.
AFAIK we don't need Symbols support in these two methods.

The second issue is that the code above is repeated twice, which increases bundle size for no gain.

Is there a way to get rid of that "symbol" support in those functions?
=====
P.S.
Again, this happened again!  I wanna my ES5 days back!
I'm certain that if the code was ES5 from the beginning we won't ever need PR's like that. :)

Babel is obviously over complicated, let's check out Bublé :) #267

It would be great is React-native-packager would behave same as node.js - use the main file from package.json.
I'm liking the Buble approach very much. @Piroro-hs would #267 resolve your issue?

I've just cloned the https://github.com/stampit-org/stampit/pull/267 repo and it works perfectly!(^^)

Great! Closing this one. :)

@Piroro-hs published stampit v3.0.4
😃


Update matchingController and questionAdapter, add getDifficultyColor function to reuse difficulty color logic across components

This pull request updates the matchingController and questionAdapter files to ensure the room has the question and the default code.
It also adds a new getDifficultyColor function to the difficulty-selector file. This function allows for the reuse of difficulty color logic across components.

Should not affect assignments, since we do not care about collaboration room in assignments


Adding one more rule to the enforcer policy sample to enable IAP to work correctly

Thanks for opening a Pull Request!
Here's a handy checklist to ensure your PR goes smoothly.

[ ] I signed Google's Contributor License Agreement
[ ] My code conforms to Google's python style.
[ ] My PR at a minimum doesn't decrease unit-test coverage (if applicable).
[ ] My PR has been functionally tested.
[ ] All of the unit-tests still pass.
[ ] Running pylint --rcfile=pylintrc passes.

These guidelines and more can be found in our contributing guidelines.

Sorry for the delay... Thanks for this, Danny! Do you think we should indicate that this example also enables IAP to work correctly? (maybe also explain why this would enable IAP to work correctly)


234 GitHub actions ci

closes #234

Some files deleted here are also deleted in PR #239

Some files deleted here are also deleted in PR #239

True. I realized this afterwards. We'll see how git manages this conflict. Shouldn't be too problematic. I'll merge this since approval has been given and deal with this while merging PR #239


Text of selected options

Hi,
is there a way to get the text of selected options?
Regards

With the below code you can get the select data JSON.
// MultiSelect Drop down select - Event
$scope.selecEvents = {
  onItemSelect: function(property) {
    console.log('select > ' +property);
  },

  onItemDeselect: function(property) {
    console.log('deselect : ' +property);
  }
}

print the property and see the text which you need and use it!! Make sure you add these events while creating multi-select!

Thank you @rajkumarpb


Tests and improvements for endpoint handling

This PR adds tests that verify how information from EndpointAnnotation is applied to DCP model for local application execution. It covers full spectrum of relevant endpoint options (EndpointAnnotation.Port set/not set, EndpointAnnotation.TargetPort set/not set, proxy vs proxy-less, replicated vs not), for Executable, Project, and Container resource.
As part of this change I also made some changes to EndpointAnnotation and ApplicationExecutor to streamline the code and make it behave as expected.
Microsoft Reviewers: Open in CodeFlow

@mitchdenny @davidfowl any other thoughts? Good enough to merge?

Yes. But lets not backport this to preview 6/GA. Lets target 8.1 for this. Given we change the critical path here I want to make sure it has more bake time ;)

Yes. But lets not backport this to preview 6/GA. Lets target 8.1 for this. Given we change the critical path here I want to make sure it has more bake time ;)

Yes, that was my assumption as well. Thanks


Fixing the overlay fix

Coverage remained the same at 100.0% when pulling 127778fc263f506595a6a679e31793665ea64959 on Coenego:overlayFix into 8e1cbd26c205c9c6a7ab2174dc992cdba5ffbec6 on CUL-DigitalServices:master.

Coverage remained the same at 100.0% when pulling 127778fc263f506595a6a679e31793665ea64959 on Coenego:overlayFix into 8e1cbd26c205c9c6a7ab2174dc992cdba5ffbec6 on CUL-DigitalServices:master.

Coverage remained the same at 100.0% when pulling 1c44719529b4b2e9c762f942a8be2c3360aa0028 on Coenego:overlayFix into 8e1cbd26c205c9c6a7ab2174dc992cdba5ffbec6 on CUL-DigitalServices:master.

Followed up on the minor review and pushed into master.


Unnecessary horizontal scroll in IFrame dialog

Category
[ ] Enhancement
[x] Bug
[ ] Question
Version
Please specify what version of the library you are using: [latest]
Expected / Desired Behavior / Question
There is no scroll in IFrameDialog
Observed Behavior

broken in #118
Steps to Reproduce
Use IFrameDialog control with specified width

This will be available today in version 1.9.0.


Reap singleUse actors

singleUse actors are for internal use:

When a password reset is initiated, a singleUse actor is created and assigned the pwreset role. A 24-hour session is created for the actor, and the session's token is sent over email as part of a password reset URL.
When a form is published, a singleUse actor is created and assigned the formview role in order to provide Enketo temporary access to the form. A 15-minute session is created for the actor.
Formerly, Google Drive backups created singleUse actors. Google Drive backups have been removed.

In some cases, singleUse actors are deleted:

If an actor with a pwreset assignment is used to change a password, the actor is then "consumed" (Actors.consume()).
All singleUse actors created for Google Drive backups should be deleted now that that functionality has been removed.

However, I think that in other cases, singleUse actors are not deleted:

24 hours elapse before a password reset token is used. In that case, a new password reset must be requested. Yet the old singleUse actor is never deleted.
Actors with a formview assignment are never deleted.

Looking at the QA server, I see:

A smaller number of non-deleted actors with a pwreset assignment
A much larger number of non-deleted actors with a formview assignment. A larger number makes sense given that every form will be associated with at least one formview actor. (It could be more than one if a request to Enketo fails.)
I see a number of singleUse actors that are not deleted, yet do not have an assignment. In all cases, their displayName contains "Enketo sync token". I assume these correspond to deleted forms.
Some deleted singleUse actors correspond to Google Drive backups. That's expected.

Given that these singleUse actors are tightly coupled with a single session, I think we should consider deleting them when we reap sessions (lib/task/reap-sessions.js). Specifically, if a session is expired, and the session was for a singleUse actor, then delete both the session and the actor.
We could also consider writing a migration to delete existing singleUse actors that are unneeded. To me, that seems lower-priority but still worthwhile.

Is there any other record that a single-use actor has been created?  If not, could they be important for audit purposes?


Issue with Splash returning multiple html snapshots

Hi,
I'm trying to return multiple html pages using splash scripts (in a single response, as in the documentation) and extracting the links from them. But I found that in the response.text and response.body, the html content is morphed whenever there's more than one page returned.  This is not the case with response.data which works fine. Why is this happening?
I tried this on the same code (and website) as in the documentation- http://splash.readthedocs.io/en/stable/scripting-ref.html#splash-html (from the later part, from the example for multiple html snapshots)
This is my splash request-->
 yield Request(url= url, callback= self.parse, dont_filter=True, meta={
     'splash': {
          'args': {'lua_source': self.lua_source, 'timeout': 200},
          'endpoint': 'execute',
       },
 })

The lua script looks like this -->
  function page_info(splash, url)
      splash:go(url)
      local res = {
        html=splash:html(),
       }
      return res
  end

  function main(splash, args)
      local base = "https://news.ycombinator.com/news?p="
      local result = treat.as_array({})
      for i=1,3 do
        local url =  base .. i
        result[i] = page_info(splash, url)
      end
      return result
  end

The resulting response:
response.data -->
{u'html': u'<html op="news"><head><meta name="referrer" content="origin"><meta name="viewport" content="width=device-width, initial-scale=1.0"><link rel="stylesheet" type="text/css" href="news.css?i7azI8MkFRPfcPhHQ7HD">\n            <link rel="shortcut icon" href="favicon.ico">\n          <link rel="alternate" type="application/rss+xml" title="RSS" href="rss">\n        <title>Hacker News</title>
response.text and response.body -->
u'[{"html": "<html op=\\"news\\"><head><meta name=\\"referrer\\" content=\\"origin\\"><meta name=\\"viewport\\" content=\\"width=device-width, initial-scale=1.0\\"><link rel=\\"stylesheet\\" type=\\"text/css\\" href=\\"news.css?i7azI8MkFRPfcPhHQ7HD\\">\\n            <link rel=\\"shortcut icon\\" href=\\"favicon.ico\\">\\n          <link rel=\\"alternate\\" type=\\"application/rss+xml\\" title=\\"RSS\\" href=\\"rss\\">\\n        <title>Hacker News</title></head>
Notice the extra \ \ in the second case. These might be escape characters or something, but they're messing with the LinkExtractor which uses the response.text and is resulting in broken links. Again, this happens only when I'm returning an array of html responses.
What am I missing here?

@rohitkoganti I have the same issue. I see you closed the ticket. Were you able to figure it out?


Follow blockchain head over http/ws

What was wrong?
Introduce Bridge type that will facilitate requesting/receiving data from a trusted provider and pumping it into the network. Currently, all it does is listen for new headers to build up the local macc, and then offer them to the network.
Thought about combining bridge functionality w/ header oracle, but eventually ended up on separating b/c ....

non-bridge nodes will still need to listen to header gossip
bridge nodes will listen to header gossip & suck new data from trusted providers

This is sort of difficult to test. We don't want to introduce a reliance on infura in our tests. I feel the same way about our geth clients. Most of this logic has well-unit-tested components. "bridge" mode is still explicitly experimental. I could be convinced to use our geth clients in CI to test this logic, but I'm not sure it's necessary. Idk, happy to field other opinions on this.
How was it fixed?

Introduced Bridge type.
Reorganized HeaderOracle into its own file.

To-Do

[ ] Add entry to the release notes (may forgo for trivial changes)
[ ] Clean up commit history

Closing this for now since this functionality will no longer be needed to keep your macc up to date, since macc's will be frozen at the merge block


Adaptability added to 2nd step

Before
900

690

601

600 less

After
900

690

601

390

375

Screenshots have been added


luci-theme-bootstrap: fix scrollbar in modal window

Before:

After:

There's something wrong with your change. Have a look at the diff I think you have changed the tabs to spaces


Feature: Add theForeman as a backend

Description of the feature
This is a very interesting tool from what i have been reading and im looking to over more self service, and was wondering if some connection to theForeman tool to run the ansible role against a specific server would be interesting.
Describe the solution you'd like
Not sure if this is in scope, but basically a user being able to select MySQL/Docker, etc.. to deploy on there host through the ansible roles that way they dont need to wait for IT, it would just get submitted than approved by an IT member and it would auto deploy through theForeman API...
Note: i have not installed this product, i am just reading up on it right now. As i dont use RedHat automation or AWS

Hi,
It's not in our scope to connect Squest to other backend than AWX/RHAAP.
That being said, if the "theForeman" has an api you can discuss with it via squest (by having an AWX in the middle anyway 😃).